-
-
Notifications
You must be signed in to change notification settings - Fork 353
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rule proposal: require-css-escape
#2318
Labels
Comments
Accepted |
To reduce code churn, maybe it should have an option to limit it to attribute selectors as indicated by const className = 'foo';
document.querySelectorAll(`.${classname}`); |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
CSS.escape should always be used when interpolating arbritrary variables into a CSS selector to ensure correct escaping.
Fail
Pass
Any tagged template should not trigger the rule:
The text was updated successfully, but these errors were encountered: