Problem/Motivation
Currently the descriptions are being fed through the default XSS filter for security. That works, however it makes the descriptions provided by the module less flexible then the core field descriptions, because core has their own list of allowed tags.
Proposed resolution
Replace XSS references with FieldFilteredMarkup, which is the class the core field descriptions use to filter their markup. This way the module always stays in sync with what core would allow.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3081655-3.patch | 3.36 KB | yogeshmpawar |
| better_field_descriptions-match-core-field-sanitization.patch | 3.3 KB | jacobbell84 |
Comments
Comment #2
yogeshmpawarReviewing the patch
Comment #3
yogeshmpawarThanks @jacobbell84 for the patch but your patch failed to apply on 8.x-1.x branch so I have updated the patch as per 8.x-1.x branch.
Comment #5
yogeshmpawarCommitted & Pushed to 8.x-1.x branch.
Comment #7
jacobbell84 commented