Opened 7 years ago
Last modified 7 years ago
#41874 new enhancement
Code Editor: Extend CSS linter to check for safecss-forbidden properties
Reported by: | westonruter | Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | needs-patch |
Focuses: | Cc: |
Description
While there is a custom HTMLHint rule which checks for Kses violations, this does not extend to CSSLint and safecss_filter_attr()
. It would be a nice enhancement to get illegal style properties flagged as lint errors as well.
Originally https://github.com/WordPress/better-code-editing/issues/60:
The
safecss_filter_attr()
function is used by KSES to filter HTMLstyle
attributes, removing any properties that are illegal. This should be applied to CSS andstyle
attributes in HTML, so that any illegal properties are flagged as such.
It seems that HTMLHint is not currently applying CSSLint to
style
attributes, but if it did, then akses
rule added to CSSLint could then apply to both HTML and CSS.
This is for the code editor introduced in [41376] for #12423.
Bumping priority to high for visibility and alignment with 4.9 goals, and given proximity to beta 1 deadline.