sindresorhus/eslint-plugin-unicorn
Rule proposal: `require-css-escape`
クローズ
#2,318 opened on 2024/04/10
help wantednew rule
Repository metrics
- Stars
- (5,022 個のスター)
- PR merge metrics
- (平均マージ 4h 30m) (30d で 26 merged PRs)
説明
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}`);