$ cnpm install @hint/hint-leading-dot-classlist
classList.add or classList.removeThis hint informs users that they should to use
Element.classList argument without a leading '.' as it
may lead to unintended results.
When writing selectors either in CSS or using DOM methods like
querySelector, class names are referred to using a leading '.',
e.g. document.querySelector('.foo'). However when modifying the
classList of an element the raw class name is expected to be used
instead, e.g. element.classList.add('foo').
Unfortunately if a leading '.' is provided to the classList APIs
it will succeed without an error, treating the '.' as part of the
name itself. This typically causes selectors elsewhere in the code
to fail to match this element. Figuring out why can be tedious and
time-consuming until the typo has been found.
This hint scans JavaScript source code to check if the argument in
element.classList.add or element.classList.remove contains a
leading '.'. If so it emits a warning to help save time debugging
this subtle issue.
const element = document.getElementById('foo');
element.classList.add('.foo');
element.classList.remove('.foo');
const element = document.getElementById('foo');
element.classList.add('foo');
element.classList.remove('foo');
This package is installed automatically by webhint:
npm install hint --save-dev
To use it, activate it via the .hintrc configuration file:
{
"connector": {...},
"formatters": [...],
"parsers": [...],
"hints": {
"leading-dot-classlist": "warning"
},
...
}
Note: The recommended way of running webhint is as a devDependency of
your project.
Copyright 2013 - present © cnpmjs.org | Home |