Focused crawls are collections of frequently-updated webcrawl data from narrow (as opposed to broad or wide) web crawls, often focused on a single domain or subdomain.
Does the bug relate to non-standard syntax (e.g. SCSS, Less etc.)?
No
What did you expect to happen?
No errors to be flagged.
What actually happened (e.g. what warnings or errors did you get)?
The following error was flagged:
2:1 error Expected selector "a" to come before selector "a::-moz-focus-inner" (no-descending-specificity)
Fixing it
This could simply be fixed by adding -moz-focus-inner to the list of known pseudo-elements in keywordSets.js, but relying on that list is obviously not the greatest—-moz-focus-inner was actually the original example given for this bug way back in #1811, and the fix still missed it.
If we instead made the selector parsing assume any selector with a double colon is a pseudo-element, not a pseudo-class, that could be a more general fix, and would match the spec:
This :: notation is introduced by the current document in order to establish a discrimination between pseudo-classes and pseudo-elements.
Of course, the confusion between e.g. :after and ::after is well-known, and the former is in (limited, but spec-compliant) use, so we would still need a keyword list, but this would solve all syntactically valid cases going forward, and existing logic would probably handle any common invalid ones gracefully.
The text was updated successfully, but these errors were encountered:
::-moz-focus-inner
is treated as a pseudo-class, not a pseudo-element, and so can cause false positives inno-descending-specificity
.no-descending-specificity
e.g.
v13.7.2
Originally encountered running it with SublimeLinter, but reproducible in the Stylelint demo page
No
No errors to be flagged.
The following error was flagged:
Fixing it
This could simply be fixed by adding
-moz-focus-inner
to the list of known pseudo-elements in keywordSets.js, but relying on that list is obviously not the greatest—-moz-focus-inner
was actually the original example given for this bug way back in #1811, and the fix still missed it.If we instead made the selector parsing assume any selector with a double colon is a pseudo-element, not a pseudo-class, that could be a more general fix, and would match the spec:
Of course, the confusion between e.g.
:after
and::after
is well-known, and the former is in (limited, but spec-compliant) use, so we would still need a keyword list, but this would solve all syntactically valid cases going forward, and existing logic would probably handle any common invalid ones gracefully.The text was updated successfully, but these errors were encountered: