sindresorhus/eslint-plugin-unicorn
Rule proposal: `require-css-escape`
Fermée
#2 318 ouverte le 10 avr. 2024
help wantednew rule
Métriques du dépôt
- Stars
- (5 022 étoiles)
- Métriques de merge PR
- (Merge moyen 4h 30m) (26 PRs mergées en 30 j)
Description
Description
CSS.escape should always be used when interpolating arbritrary variables into a CSS selector to ensure correct escaping.
Fail
document.querySelector(`#${id}`);
el.querySelectorAll(`a[href^="#${hash}"]`);
Pass
document.querySelector(`#${CSS.escape(id)}`);
el.querySelectorAll(`a[href^="#${CSS.escape(hash)}"]`);
Any tagged template should not trigger the rule:
document.querySelector(cssEscape`#${id}`);