@herb-tools/linter 0.5.0 → 0.6.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/herb-lint.js +5131 -1647
- package/dist/herb-lint.js.map +1 -1
- package/dist/index.cjs +662 -145
- package/dist/index.cjs.map +1 -1
- package/dist/index.js +654 -147
- package/dist/index.js.map +1 -1
- package/dist/package.json +4 -4
- package/dist/src/cli/argument-parser.js +0 -4
- package/dist/src/cli/argument-parser.js.map +1 -1
- package/dist/src/default-rules.js +20 -0
- package/dist/src/default-rules.js.map +1 -1
- package/dist/src/linter.js +29 -4
- package/dist/src/linter.js.map +1 -1
- package/dist/src/rules/erb-no-silent-tag-in-attribute-name.js +26 -0
- package/dist/src/rules/erb-no-silent-tag-in-attribute-name.js.map +1 -0
- package/dist/src/rules/erb-prefer-image-tag-helper.js +0 -4
- package/dist/src/rules/erb-prefer-image-tag-helper.js.map +1 -1
- package/dist/src/rules/html-aria-attribute-must-be-valid.js +11 -10
- package/dist/src/rules/html-aria-attribute-must-be-valid.js.map +1 -1
- package/dist/src/rules/html-aria-label-is-well-formatted.js +33 -0
- package/dist/src/rules/html-aria-label-is-well-formatted.js.map +1 -0
- package/dist/src/rules/html-aria-level-must-be-valid.js +26 -4
- package/dist/src/rules/html-aria-level-must-be-valid.js.map +1 -1
- package/dist/src/rules/html-aria-role-heading-requires-level.js +7 -13
- package/dist/src/rules/html-aria-role-heading-requires-level.js.map +1 -1
- package/dist/src/rules/html-aria-role-must-be-valid.js +3 -3
- package/dist/src/rules/html-aria-role-must-be-valid.js.map +1 -1
- package/dist/src/rules/html-attribute-double-quotes.js +14 -4
- package/dist/src/rules/html-attribute-double-quotes.js.map +1 -1
- package/dist/src/rules/html-attribute-equals-spacing.js +24 -0
- package/dist/src/rules/html-attribute-equals-spacing.js.map +1 -0
- package/dist/src/rules/html-attribute-values-require-quotes.js +19 -8
- package/dist/src/rules/html-attribute-values-require-quotes.js.map +1 -1
- package/dist/src/rules/html-avoid-both-disabled-and-aria-disabled.js +47 -0
- package/dist/src/rules/html-avoid-both-disabled-and-aria-disabled.js.map +1 -0
- package/dist/src/rules/html-boolean-attributes-no-value.js +9 -2
- package/dist/src/rules/html-boolean-attributes-no-value.js.map +1 -1
- package/dist/src/rules/html-iframe-has-title.js +39 -0
- package/dist/src/rules/html-iframe-has-title.js.map +1 -0
- package/dist/src/rules/html-img-require-alt.js +0 -4
- package/dist/src/rules/html-img-require-alt.js.map +1 -1
- package/dist/src/rules/html-navigation-has-label.js +43 -0
- package/dist/src/rules/html-navigation-has-label.js.map +1 -0
- package/dist/src/rules/html-no-aria-hidden-on-focusable.js +67 -0
- package/dist/src/rules/html-no-aria-hidden-on-focusable.js.map +1 -0
- package/dist/src/rules/html-no-duplicate-attributes.js +22 -25
- package/dist/src/rules/html-no-duplicate-attributes.js.map +1 -1
- package/dist/src/rules/html-no-duplicate-ids.js +2 -2
- package/dist/src/rules/html-no-duplicate-ids.js.map +1 -1
- package/dist/src/rules/html-no-empty-headings.js +0 -21
- package/dist/src/rules/html-no-empty-headings.js.map +1 -1
- package/dist/src/rules/html-no-positive-tab-index.js +21 -0
- package/dist/src/rules/html-no-positive-tab-index.js.map +1 -0
- package/dist/src/rules/html-no-self-closing.js +22 -0
- package/dist/src/rules/html-no-self-closing.js.map +1 -0
- package/dist/src/rules/html-no-title-attribute.js +27 -0
- package/dist/src/rules/html-no-title-attribute.js.map +1 -0
- package/dist/src/rules/html-tag-name-lowercase.js +35 -23
- package/dist/src/rules/html-tag-name-lowercase.js.map +1 -1
- package/dist/src/rules/index.js +10 -0
- package/dist/src/rules/index.js.map +1 -1
- package/dist/src/rules/rule-utils.js +176 -22
- package/dist/src/rules/rule-utils.js.map +1 -1
- package/dist/src/rules/svg-tag-name-capitalization.js +0 -8
- package/dist/src/rules/svg-tag-name-capitalization.js.map +1 -1
- package/dist/src/types.js.map +1 -1
- package/dist/tsconfig.tsbuildinfo +1 -1
- package/dist/types/cli/index.d.ts +4 -0
- package/dist/types/rules/erb-no-silent-tag-in-attribute-name.d.ts +7 -0
- package/dist/types/rules/html-aria-label-is-well-formatted.d.ts +7 -0
- package/dist/types/rules/html-attribute-equals-spacing.d.ts +7 -0
- package/dist/types/rules/html-avoid-both-disabled-and-aria-disabled.d.ts +7 -0
- package/dist/types/rules/html-iframe-has-title.d.ts +7 -0
- package/dist/types/rules/html-navigation-has-label.d.ts +7 -0
- package/dist/types/rules/html-no-aria-hidden-on-focusable.d.ts +7 -0
- package/dist/types/rules/html-no-positive-tab-index.d.ts +7 -0
- package/dist/types/rules/html-no-self-closing.d.ts +7 -0
- package/dist/types/rules/html-no-title-attribute.d.ts +7 -0
- package/dist/types/rules/html-tag-name-lowercase.d.ts +2 -1
- package/dist/types/rules/index.d.ts +10 -0
- package/dist/types/rules/rule-utils.d.ts +107 -13
- package/dist/types/src/rules/erb-no-silent-tag-in-attribute-name.d.ts +7 -0
- package/dist/types/src/rules/html-aria-label-is-well-formatted.d.ts +7 -0
- package/dist/types/src/rules/html-attribute-equals-spacing.d.ts +7 -0
- package/dist/types/src/rules/html-avoid-both-disabled-and-aria-disabled.d.ts +7 -0
- package/dist/types/src/rules/html-iframe-has-title.d.ts +7 -0
- package/dist/types/src/rules/html-navigation-has-label.d.ts +7 -0
- package/dist/types/src/rules/html-no-aria-hidden-on-focusable.d.ts +7 -0
- package/dist/types/src/rules/html-no-positive-tab-index.d.ts +7 -0
- package/dist/types/src/rules/html-no-self-closing.d.ts +7 -0
- package/dist/types/src/rules/html-no-title-attribute.d.ts +7 -0
- package/dist/types/src/rules/html-tag-name-lowercase.d.ts +2 -1
- package/dist/types/src/rules/index.d.ts +10 -0
- package/dist/types/src/rules/rule-utils.d.ts +107 -13
- package/dist/types/src/types.d.ts +24 -0
- package/dist/types/types.d.ts +24 -0
- package/docs/rules/README.md +12 -2
- package/docs/rules/erb-no-silent-tag-in-attribute-name.md +34 -0
- package/docs/rules/html-aria-label-is-well-formatted.md +49 -0
- package/docs/rules/html-attribute-equals-spacing.md +35 -0
- package/docs/rules/html-avoid-both-disabled-and-aria-disabled.md +48 -0
- package/docs/rules/html-iframe-has-title.md +43 -0
- package/docs/rules/html-navigation-has-label.md +61 -0
- package/docs/rules/html-no-aria-hidden-on-focusable.md +54 -0
- package/docs/rules/html-no-positive-tab-index.md +55 -0
- package/docs/rules/html-no-self-closing.md +65 -0
- package/docs/rules/html-no-title-attribute.md +69 -0
- package/docs/rules/html-tag-name-lowercase.md +16 -3
- package/package.json +4 -4
- package/src/cli/argument-parser.ts +0 -5
- package/src/default-rules.ts +20 -0
- package/src/linter.ts +30 -4
- package/src/rules/erb-no-silent-tag-in-attribute-name.ts +40 -0
- package/src/rules/erb-prefer-image-tag-helper.ts +2 -7
- package/src/rules/html-aria-attribute-must-be-valid.ts +28 -32
- package/src/rules/html-aria-label-is-well-formatted.ts +59 -0
- package/src/rules/html-aria-level-must-be-valid.ts +38 -5
- package/src/rules/html-aria-role-heading-requires-level.ts +16 -28
- package/src/rules/html-aria-role-must-be-valid.ts +5 -5
- package/src/rules/html-attribute-double-quotes.ts +21 -6
- package/src/rules/html-attribute-equals-spacing.ts +41 -0
- package/src/rules/html-attribute-values-require-quotes.ts +29 -9
- package/src/rules/html-avoid-both-disabled-and-aria-disabled.ts +66 -0
- package/src/rules/html-boolean-attributes-no-value.ts +17 -4
- package/src/rules/html-iframe-has-title.ts +62 -0
- package/src/rules/html-img-require-alt.ts +2 -7
- package/src/rules/html-navigation-has-label.ts +64 -0
- package/src/rules/html-no-aria-hidden-on-focusable.ts +90 -0
- package/src/rules/html-no-duplicate-attributes.ts +28 -28
- package/src/rules/html-no-duplicate-ids.ts +4 -3
- package/src/rules/html-no-empty-headings.ts +2 -31
- package/src/rules/html-no-positive-tab-index.ts +33 -0
- package/src/rules/html-no-self-closing.ts +36 -0
- package/src/rules/html-no-title-attribute.ts +42 -0
- package/src/rules/html-tag-name-lowercase.ts +42 -29
- package/src/rules/index.ts +10 -0
- package/src/rules/rule-utils.ts +260 -39
- package/src/rules/svg-tag-name-capitalization.ts +2 -9
- package/src/types.ts +27 -0
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-aria-level-must-be-valid.js","sourceRoot":"","sources":["../../../src/rules/html-aria-level-must-be-valid.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-aria-level-must-be-valid.js","sourceRoot":"","sources":["../../../src/rules/html-aria-level-must-be-valid.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,qBAAqB,EAAuE,MAAM,iBAAiB,CAAA;AAC5H,OAAO,EAAE,2BAA2B,EAAE,YAAY,EAAE,kBAAkB,EAAE,qBAAqB,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKxI,MAAM,+BAAgC,SAAQ,qBAAqB;IACvD,+BAA+B,CAAC,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAoC;QAC1H,IAAI,aAAa,KAAK,YAAY;YAAE,OAAM;QAE1C,IAAI,CAAC,iBAAiB,CAAC,cAAc,EAAE,aAAa,CAAC,CAAA;IACvD,CAAC;IAES,gCAAgC,CAAC,EAAE,aAAa,EAAE,UAAU,EAAE,aAAa,EAAqC;QACxH,IAAI,aAAa,KAAK,YAAY;YAAE,OAAM;QAE1C,MAAM,kBAAkB,GAAG,2BAA2B,CAAC,UAAU,CAAC,CAAA;QAElE,IAAI,kBAAkB,KAAK,IAAI,EAAE,CAAC;YAChC,IAAI,CAAC,iBAAiB,CAAC,kBAAkB,EAAE,aAAa,CAAC,CAAA;YACzD,OAAM;QACR,CAAC;QAED,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC;YAAE,OAAM;QAErC,MAAM,YAAY,GAAG,kBAAkB,CAAC,UAAU,CAAC,CAAA;QACnD,MAAM,cAAc,GAAG,qBAAqB,CAAC,UAAU,CAAC,CAAC,MAAM,CAAC,eAAe,CAAC,CAAA;QAEhF,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,IAAI,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACzD,MAAM,UAAU,GAAG,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;YAElE,yDAAyD;YACzD,MAAM,OAAO,GAAG,cAAc,CAAC,CAAC,CAAC,CAAA;YACjC,MAAM,OAAO,GAAG,GAAG,OAAO,CAAC,WAAW,EAAE,KAAK,IAAI,EAAE,GAAG,OAAO,CAAC,OAAO,EAAE,KAAK,IAAI,EAAE,GAAG,OAAO,CAAC,WAAW,EAAE,KAAK,IAAI,EAAE,EAAE,CAAA;YAEvH,IAAI,CAAC,UAAU,CACb,0EAA0E,UAAU,+BAA+B,OAAO,KAAK,EAC/H,aAAa,CAAC,QAAQ,CACvB,CAAA;QACH,CAAC;IACH,CAAC;IAEO,iBAAiB,CAAC,cAAsB,EAAE,aAAgC;QAChF,IAAI,CAAC,cAAc,IAAI,cAAc,KAAK,EAAE,EAAE,CAAC;YAC7C,IAAI,CAAC,UAAU,CACb,sFAAsF,EACtF,aAAa,CAAC,QAAQ,CACvB,CAAA;YAED,OAAM;QACR,CAAC;QAED,MAAM,MAAM,GAAG,QAAQ,CAAC,cAAc,CAAC,CAAA;QAEvC,IAAI,KAAK,CAAC,MAAM,CAAC,IAAI,MAAM,GAAG,CAAC,IAAI,MAAM,GAAG,CAAC,IAAI,cAAc,KAAK,MAAM,CAAC,QAAQ,EAAE,EAAE,CAAC;YACtF,IAAI,CAAC,UAAU,CACb,0EAA0E,cAAc,KAAK,EAC7F,aAAa,CAAC,QAAQ,CACvB,CAAA;QACH,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,4BAA6B,SAAQ,UAAU;IAC1D,IAAI,GAAG,+BAA+B,CAAA;IAEtC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,+BAA+B,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEvE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -1,19 +1,13 @@
|
|
|
1
|
-
import { AttributeVisitorMixin, getAttributeName, getAttributes } from "./rule-utils.js";
|
|
2
1
|
import { ParserRule } from "../types.js";
|
|
2
|
+
import { AttributeVisitorMixin, getAttributeName, getAttributes } from "./rule-utils.js";
|
|
3
3
|
class AriaRoleHeadingRequiresLevel extends AttributeVisitorMixin {
|
|
4
|
-
|
|
5
|
-
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
if (
|
|
4
|
+
checkStaticAttributeStaticValue({ attributeName, attributeValue, attributeNode, parentNode }) {
|
|
5
|
+
if (!(attributeName === "role" && attributeValue === "heading"))
|
|
6
|
+
return;
|
|
7
|
+
const ariaLevelAttributes = getAttributes(parentNode).find(attribute => getAttributeName(attribute) === "aria-level");
|
|
8
|
+
if (ariaLevelAttributes)
|
|
9
9
|
return;
|
|
10
|
-
|
|
11
|
-
const allAttributes = getAttributes(parentNode);
|
|
12
|
-
// If we have a role="heading", we must check for aria-level
|
|
13
|
-
const ariaLevelAttr = allAttributes.find(attr => getAttributeName(attr) === "aria-level");
|
|
14
|
-
if (!ariaLevelAttr) {
|
|
15
|
-
this.addOffense(`Element with \`role="heading"\` must have an \`aria-level\` attribute.`, attributeNode.location, "error");
|
|
16
|
-
}
|
|
10
|
+
this.addOffense(`Element with \`role="heading"\` must have an \`aria-level\` attribute.`, attributeNode.location, "error");
|
|
17
11
|
}
|
|
18
12
|
}
|
|
19
13
|
export class HTMLAriaRoleHeadingRequiresLevelRule extends ParserRule {
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-aria-role-heading-requires-level.js","sourceRoot":"","sources":["../../../src/rules/html-aria-role-heading-requires-level.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-aria-role-heading-requires-level.js","sourceRoot":"","sources":["../../../src/rules/html-aria-role-heading-requires-level.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,qBAAqB,EAAE,gBAAgB,EAAE,aAAa,EAAoC,MAAM,iBAAiB,CAAA;AAK1H,MAAM,4BAA6B,SAAQ,qBAAqB;IACpD,+BAA+B,CAAC,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAE,UAAU,EAAoC;QACtI,IAAI,CAAC,CAAC,aAAa,KAAK,MAAM,IAAI,cAAc,KAAK,SAAS,CAAC;YAAE,OAAM;QAEvE,MAAM,mBAAmB,GAAG,aAAa,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC,gBAAgB,CAAC,SAAS,CAAC,KAAK,YAAY,CAAC,CAAA;QAErH,IAAI,mBAAmB;YAAE,OAAM;QAE/B,IAAI,CAAC,UAAU,CACb,wEAAwE,EACxE,aAAa,CAAC,QAAQ,EACtB,OAAO,CACR,CAAA;IACH,CAAC;CACF;AAED,MAAM,OAAO,oCAAqC,SAAQ,UAAU;IAClE,IAAI,GAAG,uCAAuC,CAAA;IAE9C,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,4BAA4B,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEpE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -1,10 +1,10 @@
|
|
|
1
|
-
import { AttributeVisitorMixin, VALID_ARIA_ROLES } from "./rule-utils.js";
|
|
2
1
|
import { ParserRule } from "../types.js";
|
|
2
|
+
import { AttributeVisitorMixin, VALID_ARIA_ROLES } from "./rule-utils.js";
|
|
3
3
|
class AriaRoleMustBeValid extends AttributeVisitorMixin {
|
|
4
|
-
|
|
4
|
+
checkStaticAttributeStaticValue({ attributeName, attributeValue, attributeNode }) {
|
|
5
5
|
if (attributeName !== "role")
|
|
6
6
|
return;
|
|
7
|
-
if (attributeValue
|
|
7
|
+
if (!attributeValue)
|
|
8
8
|
return;
|
|
9
9
|
if (VALID_ARIA_ROLES.has(attributeValue))
|
|
10
10
|
return;
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-aria-role-must-be-valid.js","sourceRoot":"","sources":["../../../src/rules/html-aria-role-must-be-valid.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-aria-role-must-be-valid.js","sourceRoot":"","sources":["../../../src/rules/html-aria-role-must-be-valid.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,qBAAqB,EAAE,gBAAgB,EAAoC,MAAM,iBAAiB,CAAA;AAK3G,MAAM,mBAAoB,SAAQ,qBAAqB;IAC3C,+BAA+B,CAAC,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAoC;QAC1H,IAAI,aAAa,KAAK,MAAM;YAAE,OAAM;QACpC,IAAI,CAAC,cAAc;YAAE,OAAM;QAC3B,IAAI,gBAAgB,CAAC,GAAG,CAAC,cAAc,CAAC;YAAE,OAAM;QAEhD,IAAI,CAAC,UAAU,CACb,4DAA4D,cAAc,uBAAuB,EACjG,aAAa,CAAC,QAAQ,EACtB,OAAO,CACR,CAAA;IACH,CAAC;CACF;AAED,MAAM,OAAO,2BAA4B,SAAQ,UAAU;IACzD,IAAI,GAAG,8BAA8B,CAAA;IAErC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,mBAAmB,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAE3D,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -1,14 +1,24 @@
|
|
|
1
|
-
import { AttributeVisitorMixin, getAttributeValueQuoteType, hasAttributeValue } from "./rule-utils.js";
|
|
2
1
|
import { ParserRule } from "../types.js";
|
|
2
|
+
import { AttributeVisitorMixin, getAttributeValueQuoteType, hasAttributeValue } from "./rule-utils.js";
|
|
3
|
+
import { filterLiteralNodes } from "@herb-tools/core";
|
|
3
4
|
class AttributeDoubleQuotesVisitor extends AttributeVisitorMixin {
|
|
4
|
-
|
|
5
|
+
checkStaticAttributeStaticValue({ attributeName, attributeValue, attributeNode }) {
|
|
5
6
|
if (!hasAttributeValue(attributeNode))
|
|
6
7
|
return;
|
|
7
8
|
if (getAttributeValueQuoteType(attributeNode) !== "single")
|
|
8
9
|
return;
|
|
9
10
|
if (attributeValue?.includes('"'))
|
|
10
|
-
return;
|
|
11
|
-
this.addOffense(`Attribute \`${attributeName}\` uses single quotes. Prefer double quotes for HTML attribute values: \`${attributeName}="
|
|
11
|
+
return;
|
|
12
|
+
this.addOffense(`Attribute \`${attributeName}\` uses single quotes. Prefer double quotes for HTML attribute values: \`${attributeName}="${attributeValue}"\`.`, attributeNode.value.location, "warning");
|
|
13
|
+
}
|
|
14
|
+
checkStaticAttributeDynamicValue({ attributeName, valueNodes, attributeNode, combinedValue }) {
|
|
15
|
+
if (!hasAttributeValue(attributeNode))
|
|
16
|
+
return;
|
|
17
|
+
if (getAttributeValueQuoteType(attributeNode) !== "single")
|
|
18
|
+
return;
|
|
19
|
+
if (filterLiteralNodes(valueNodes).some(node => node.content?.includes('"')))
|
|
20
|
+
return;
|
|
21
|
+
this.addOffense(`Attribute \`${attributeName}\` uses single quotes. Prefer double quotes for HTML attribute values: \`${attributeName}="${combinedValue}"\`.`, attributeNode.value.location, "warning");
|
|
12
22
|
}
|
|
13
23
|
}
|
|
14
24
|
export class HTMLAttributeDoubleQuotesRule extends ParserRule {
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-attribute-double-quotes.js","sourceRoot":"","sources":["../../../src/rules/html-attribute-double-quotes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-attribute-double-quotes.js","sourceRoot":"","sources":["../../../src/rules/html-attribute-double-quotes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,qBAAqB,EAAuE,0BAA0B,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAA;AAC3K,OAAO,EAAE,kBAAkB,EAAE,MAAM,kBAAkB,CAAA;AAKrD,MAAM,4BAA6B,SAAQ,qBAAqB;IACpD,+BAA+B,CAAC,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAoC;QAC1H,IAAI,CAAC,iBAAiB,CAAC,aAAa,CAAC;YAAE,OAAM;QAC7C,IAAI,0BAA0B,CAAC,aAAa,CAAC,KAAK,QAAQ;YAAE,OAAM;QAClE,IAAI,cAAc,EAAE,QAAQ,CAAC,GAAG,CAAC;YAAE,OAAM;QAEzC,IAAI,CAAC,UAAU,CACb,eAAe,aAAa,4EAA4E,aAAa,KAAK,cAAc,MAAM,EAC9I,aAAa,CAAC,KAAM,CAAC,QAAQ,EAC7B,SAAS,CACV,CAAA;IACH,CAAC;IAES,gCAAgC,CAAC,EAAE,aAAa,EAAE,UAAU,EAAE,aAAa,EAAE,aAAa,EAAqC;QACvI,IAAI,CAAC,iBAAiB,CAAC,aAAa,CAAC;YAAE,OAAM;QAC7C,IAAI,0BAA0B,CAAC,aAAa,CAAC,KAAK,QAAQ;YAAE,OAAM;QAClE,IAAI,kBAAkB,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,CAAC,OAAO,EAAE,QAAQ,CAAC,GAAG,CAAC,CAAC;YAAE,OAAM;QAEpF,IAAI,CAAC,UAAU,CACb,eAAe,aAAa,4EAA4E,aAAa,KAAK,aAAa,MAAM,EAC7I,aAAa,CAAC,KAAM,CAAC,QAAQ,EAC7B,SAAS,CACV,CAAA;IACH,CAAC;CACF;AAED,MAAM,OAAO,6BAA8B,SAAQ,UAAU;IAC3D,IAAI,GAAG,8BAA8B,CAAA;IAErC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,4BAA4B,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEpE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -0,0 +1,24 @@
|
|
|
1
|
+
import { BaseRuleVisitor } from "./rule-utils.js";
|
|
2
|
+
import { ParserRule } from "../types.js";
|
|
3
|
+
class HTMLAttributeEqualsSpacingVisitor extends BaseRuleVisitor {
|
|
4
|
+
visitHTMLAttributeNode(attribute) {
|
|
5
|
+
if (!attribute.equals || !attribute.name || !attribute.value) {
|
|
6
|
+
return;
|
|
7
|
+
}
|
|
8
|
+
if (attribute.equals.value.startsWith(" ")) {
|
|
9
|
+
this.addOffense("Remove whitespace before `=` in HTML attribute", attribute.equals.location, "error");
|
|
10
|
+
}
|
|
11
|
+
if (attribute.equals.value.endsWith(" ")) {
|
|
12
|
+
this.addOffense("Remove whitespace after `=` in HTML attribute", attribute.equals.location, "error");
|
|
13
|
+
}
|
|
14
|
+
}
|
|
15
|
+
}
|
|
16
|
+
export class HTMLAttributeEqualsSpacingRule extends ParserRule {
|
|
17
|
+
name = "html-attribute-equals-spacing";
|
|
18
|
+
check(result, context) {
|
|
19
|
+
const visitor = new HTMLAttributeEqualsSpacingVisitor(this.name, context);
|
|
20
|
+
visitor.visit(result.value);
|
|
21
|
+
return visitor.offenses;
|
|
22
|
+
}
|
|
23
|
+
}
|
|
24
|
+
//# sourceMappingURL=html-attribute-equals-spacing.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"html-attribute-equals-spacing.js","sourceRoot":"","sources":["../../../src/rules/html-attribute-equals-spacing.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AACjD,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AAKxC,MAAM,iCAAkC,SAAQ,eAAe;IAC7D,sBAAsB,CAAC,SAA4B;QACjD,IAAI,CAAC,SAAS,CAAC,MAAM,IAAI,CAAC,SAAS,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,KAAK,EAAE,CAAC;YAC7D,OAAM;QACR,CAAC;QAED,IAAI,SAAS,CAAC,MAAM,CAAC,KAAK,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YAC3C,IAAI,CAAC,UAAU,CACb,gDAAgD,EAChD,SAAS,CAAC,MAAM,CAAC,QAAQ,EACzB,OAAO,CACR,CAAA;QACH,CAAC;QAED,IAAI,SAAS,CAAC,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;YACzC,IAAI,CAAC,UAAU,CACb,+CAA+C,EAC/C,SAAS,CAAC,MAAM,CAAC,QAAQ,EACzB,OAAO,CACR,CAAA;QACH,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,8BAA+B,SAAQ,UAAU;IAC5D,IAAI,GAAG,+BAA+B,CAAA;IAEtC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,iCAAiC,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEzE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -1,15 +1,26 @@
|
|
|
1
|
-
import { AttributeVisitorMixin } from "./rule-utils.js";
|
|
2
1
|
import { ParserRule } from "../types.js";
|
|
2
|
+
import { AttributeVisitorMixin } from "./rule-utils.js";
|
|
3
3
|
class AttributeValuesRequireQuotesVisitor extends AttributeVisitorMixin {
|
|
4
|
-
|
|
5
|
-
if (attributeNode
|
|
4
|
+
checkStaticAttributeStaticValue({ attributeName, attributeValue, attributeNode }) {
|
|
5
|
+
if (this.hasAttributeValue(attributeNode))
|
|
6
6
|
return;
|
|
7
|
-
|
|
8
|
-
|
|
7
|
+
if (this.isQuoted(attributeNode))
|
|
8
|
+
return;
|
|
9
|
+
this.addOffense(`Attribute value should be quoted: \`${attributeName}="${attributeValue}"\`. Always wrap attribute values in quotes.`, attributeNode.value.location, "error");
|
|
10
|
+
}
|
|
11
|
+
checkStaticAttributeDynamicValue({ attributeName, attributeNode, combinedValue }) {
|
|
12
|
+
if (this.hasAttributeValue(attributeNode))
|
|
9
13
|
return;
|
|
10
|
-
this.
|
|
11
|
-
|
|
12
|
-
`Attribute value should be quoted: \`${attributeName}="
|
|
14
|
+
if (this.isQuoted(attributeNode))
|
|
15
|
+
return;
|
|
16
|
+
this.addOffense(`Attribute value should be quoted: \`${attributeName}="${combinedValue}"\`. Always wrap attribute values in quotes.`, attributeNode.value.location, "error");
|
|
17
|
+
}
|
|
18
|
+
hasAttributeValue(attributeNode) {
|
|
19
|
+
return attributeNode.value?.type !== "AST_HTML_ATTRIBUTE_VALUE_NODE";
|
|
20
|
+
}
|
|
21
|
+
isQuoted(attributeNode) {
|
|
22
|
+
const valueNode = attributeNode.value;
|
|
23
|
+
return valueNode.quoted;
|
|
13
24
|
}
|
|
14
25
|
}
|
|
15
26
|
export class HTMLAttributeValuesRequireQuotesRule extends ParserRule {
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-attribute-values-require-quotes.js","sourceRoot":"","sources":["../../../src/rules/html-attribute-values-require-quotes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-attribute-values-require-quotes.js","sourceRoot":"","sources":["../../../src/rules/html-attribute-values-require-quotes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,qBAAqB,EAAuE,MAAM,iBAAiB,CAAA;AAK5H,MAAM,mCAAoC,SAAQ,qBAAqB;IAC3D,+BAA+B,CAAC,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAoC;QAC1H,IAAI,IAAI,CAAC,iBAAiB,CAAC,aAAa,CAAC;YAAE,OAAM;QACjD,IAAI,IAAI,CAAC,QAAQ,CAAC,aAAa,CAAC;YAAE,OAAM;QAExC,IAAI,CAAC,UAAU,CACb,uCAAuC,aAAa,KAAK,cAAc,8CAA8C,EACrH,aAAa,CAAC,KAAM,CAAC,QAAQ,EAC7B,OAAO,CACR,CAAA;IACH,CAAC;IAES,gCAAgC,CAAC,EAAE,aAAa,EAAE,aAAa,EAAE,aAAa,EAAqC;QAC3H,IAAI,IAAI,CAAC,iBAAiB,CAAC,aAAa,CAAC;YAAE,OAAM;QACjD,IAAI,IAAI,CAAC,QAAQ,CAAC,aAAa,CAAC;YAAE,OAAM;QAExC,IAAI,CAAC,UAAU,CACb,uCAAuC,aAAa,KAAK,aAAa,8CAA8C,EACpH,aAAa,CAAC,KAAM,CAAC,QAAQ,EAC7B,OAAO,CACR,CAAA;IACH,CAAC;IAEO,iBAAiB,CAAC,aAAgC;QACxD,OAAO,aAAa,CAAC,KAAK,EAAE,IAAI,KAAK,+BAA+B,CAAA;IACtE,CAAC;IAEO,QAAQ,CAAC,aAAgC;QAC/C,MAAM,SAAS,GAAG,aAAa,CAAC,KAA+B,CAAA;QAE/D,OAAO,SAAS,CAAC,MAAM,CAAA;IACzB,CAAC;CACF;AAED,MAAM,OAAO,oCAAqC,SAAQ,UAAU;IAClE,IAAI,GAAG,sCAAsC,CAAA;IAE7C,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,mCAAmC,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAE3E,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -0,0 +1,47 @@
|
|
|
1
|
+
import { ParserRule } from "../types.js";
|
|
2
|
+
import { BaseRuleVisitor, getTagName, hasAttribute, getAttributes, findAttributeByName } from "./rule-utils.js";
|
|
3
|
+
const ELEMENTS_WITH_NATIVE_DISABLED_ATTRIBUTE_SUPPORT = new Set([
|
|
4
|
+
"button", "fieldset", "input", "optgroup", "option", "select", "textarea"
|
|
5
|
+
]);
|
|
6
|
+
class AvoidBothDisabledAndAriaDisabledVisitor extends BaseRuleVisitor {
|
|
7
|
+
visitHTMLOpenTagNode(node) {
|
|
8
|
+
this.checkElement(node);
|
|
9
|
+
super.visitHTMLOpenTagNode(node);
|
|
10
|
+
}
|
|
11
|
+
checkElement(node) {
|
|
12
|
+
const tagName = getTagName(node);
|
|
13
|
+
if (!tagName || !ELEMENTS_WITH_NATIVE_DISABLED_ATTRIBUTE_SUPPORT.has(tagName)) {
|
|
14
|
+
return;
|
|
15
|
+
}
|
|
16
|
+
const hasDisabled = hasAttribute(node, "disabled");
|
|
17
|
+
const hasAriaDisabled = hasAttribute(node, "aria-disabled");
|
|
18
|
+
if ((hasDisabled && this.hasERBContent(node, "disabled")) || (hasAriaDisabled && this.hasERBContent(node, "aria-disabled"))) {
|
|
19
|
+
return;
|
|
20
|
+
}
|
|
21
|
+
if (hasDisabled && hasAriaDisabled) {
|
|
22
|
+
this.addOffense("aria-disabled may be used in place of native HTML disabled to allow tab-focus on an otherwise ignored element. Setting both attributes is contradictory and confusing. Choose either disabled or aria-disabled, not both.", node.tag_name.location, "error");
|
|
23
|
+
}
|
|
24
|
+
}
|
|
25
|
+
hasERBContent(node, attributeName) {
|
|
26
|
+
const attributes = getAttributes(node);
|
|
27
|
+
const attribute = findAttributeByName(attributes, attributeName);
|
|
28
|
+
if (!attribute)
|
|
29
|
+
return false;
|
|
30
|
+
const valueNode = attribute.value;
|
|
31
|
+
if (!valueNode || valueNode.type !== "AST_HTML_ATTRIBUTE_VALUE_NODE")
|
|
32
|
+
return false;
|
|
33
|
+
const htmlValueNode = valueNode;
|
|
34
|
+
if (!htmlValueNode.children)
|
|
35
|
+
return false;
|
|
36
|
+
return htmlValueNode.children.some((child) => child.type === "AST_ERB_CONTENT_NODE");
|
|
37
|
+
}
|
|
38
|
+
}
|
|
39
|
+
export class HTMLAvoidBothDisabledAndAriaDisabledRule extends ParserRule {
|
|
40
|
+
name = "html-avoid-both-disabled-and-aria-disabled";
|
|
41
|
+
check(result, context) {
|
|
42
|
+
const visitor = new AvoidBothDisabledAndAriaDisabledVisitor(this.name, context);
|
|
43
|
+
visitor.visit(result.value);
|
|
44
|
+
return visitor.offenses;
|
|
45
|
+
}
|
|
46
|
+
}
|
|
47
|
+
//# sourceMappingURL=html-avoid-both-disabled-and-aria-disabled.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"html-avoid-both-disabled-and-aria-disabled.js","sourceRoot":"","sources":["../../../src/rules/html-avoid-both-disabled-and-aria-disabled.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,YAAY,EAAE,aAAa,EAAE,mBAAmB,EAAE,MAAM,iBAAiB,CAAA;AAK/G,MAAM,+CAA+C,GAAG,IAAI,GAAG,CAAC;IAC9D,QAAQ,EAAE,UAAU,EAAE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,QAAQ,EAAE,UAAU;CAC1E,CAAC,CAAA;AAEF,MAAM,uCAAwC,SAAQ,eAAe;IACnE,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,CAAA;QACvB,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;IAEO,YAAY,CAAC,IAAqB;QACxC,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,CAAC,CAAA;QAEhC,IAAI,CAAC,OAAO,IAAI,CAAC,+CAA+C,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;YAC9E,OAAM;QACR,CAAC;QAED,MAAM,WAAW,GAAG,YAAY,CAAC,IAAI,EAAE,UAAU,CAAC,CAAA;QAClD,MAAM,eAAe,GAAG,YAAY,CAAC,IAAI,EAAE,eAAe,CAAC,CAAA;QAE3D,IAAI,CAAC,WAAW,IAAI,IAAI,CAAC,aAAa,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC,IAAI,CAAC,eAAe,IAAI,IAAI,CAAC,aAAa,CAAC,IAAI,EAAE,eAAe,CAAC,CAAC,EAAE,CAAC;YAC5H,OAAM;QACR,CAAC;QAED,IAAI,WAAW,IAAI,eAAe,EAAE,CAAC;YACnC,IAAI,CAAC,UAAU,CACb,2NAA2N,EAC3N,IAAI,CAAC,QAAS,CAAC,QAAQ,EACvB,OAAO,CACR,CAAA;QACH,CAAC;IACH,CAAC;IAEO,aAAa,CAAC,IAAqB,EAAE,aAAqB;QAChE,MAAM,UAAU,GAAG,aAAa,CAAC,IAAI,CAAC,CAAA;QAEtC,MAAM,SAAS,GAAG,mBAAmB,CAAC,UAAU,EAAE,aAAa,CAAC,CAAA;QAChE,IAAI,CAAC,SAAS;YAAE,OAAO,KAAK,CAAA;QAE5B,MAAM,SAAS,GAAG,SAAS,CAAC,KAAK,CAAA;QACjC,IAAI,CAAC,SAAS,IAAI,SAAS,CAAC,IAAI,KAAK,+BAA+B;YAAE,OAAO,KAAK,CAAA;QAElF,MAAM,aAAa,GAAG,SAAmC,CAAA;QACzD,IAAI,CAAC,aAAa,CAAC,QAAQ;YAAE,OAAO,KAAK,CAAA;QAEzC,OAAO,aAAa,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,KAAW,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,KAAK,sBAAsB,CAAC,CAAA;IAC5F,CAAC;CACF;AAED,MAAM,OAAO,wCAAyC,SAAQ,UAAU;IACtE,IAAI,GAAG,4CAA4C,CAAA;IAEnD,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,uCAAuC,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAE/E,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -1,13 +1,20 @@
|
|
|
1
|
-
import { AttributeVisitorMixin, isBooleanAttribute, hasAttributeValue } from "./rule-utils.js";
|
|
2
1
|
import { ParserRule } from "../types.js";
|
|
2
|
+
import { AttributeVisitorMixin, isBooleanAttribute, hasAttributeValue } from "./rule-utils.js";
|
|
3
3
|
class BooleanAttributesNoValueVisitor extends AttributeVisitorMixin {
|
|
4
|
-
|
|
4
|
+
checkStaticAttributeStaticValue({ attributeName, attributeNode }) {
|
|
5
5
|
if (!isBooleanAttribute(attributeName))
|
|
6
6
|
return;
|
|
7
7
|
if (!hasAttributeValue(attributeNode))
|
|
8
8
|
return;
|
|
9
9
|
this.addOffense(`Boolean attribute \`${attributeName}\` should not have a value. Use \`${attributeName}\` instead of \`${attributeName}="${attributeName}"\`.`, attributeNode.value.location, "error");
|
|
10
10
|
}
|
|
11
|
+
checkStaticAttributeDynamicValue({ attributeName, attributeNode, combinedValue }) {
|
|
12
|
+
if (!isBooleanAttribute(attributeName))
|
|
13
|
+
return;
|
|
14
|
+
if (!hasAttributeValue(attributeNode))
|
|
15
|
+
return;
|
|
16
|
+
this.addOffense(`Boolean attribute \`${attributeName}\` should not have a value. Use \`${attributeName}\` instead of \`${attributeName}="${combinedValue}"\`.`, attributeNode.value.location, "error");
|
|
17
|
+
}
|
|
11
18
|
}
|
|
12
19
|
export class HTMLBooleanAttributesNoValueRule extends ParserRule {
|
|
13
20
|
name = "html-boolean-attributes-no-value";
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-boolean-attributes-no-value.js","sourceRoot":"","sources":["../../../src/rules/html-boolean-attributes-no-value.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-boolean-attributes-no-value.js","sourceRoot":"","sources":["../../../src/rules/html-boolean-attributes-no-value.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,qBAAqB,EAAuE,kBAAkB,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAA;AAKnK,MAAM,+BAAgC,SAAQ,qBAAqB;IACvD,+BAA+B,CAAC,EAAE,aAAa,EAAE,aAAa,EAAoC;QAC1G,IAAI,CAAC,kBAAkB,CAAC,aAAa,CAAC;YAAE,OAAM;QAC9C,IAAI,CAAC,iBAAiB,CAAC,aAAa,CAAC;YAAE,OAAM;QAE7C,IAAI,CAAC,UAAU,CACb,uBAAuB,aAAa,qCAAqC,aAAa,mBAAmB,aAAa,KAAK,aAAa,MAAM,EAC9I,aAAa,CAAC,KAAM,CAAC,QAAQ,EAC7B,OAAO,CACR,CAAA;IACH,CAAC;IAES,gCAAgC,CAAC,EAAE,aAAa,EAAE,aAAa,EAAE,aAAa,EAAqC;QAC3H,IAAI,CAAC,kBAAkB,CAAC,aAAa,CAAC;YAAE,OAAM;QAC9C,IAAI,CAAC,iBAAiB,CAAC,aAAa,CAAC;YAAE,OAAM;QAE7C,IAAI,CAAC,UAAU,CACb,uBAAuB,aAAa,qCAAqC,aAAa,mBAAmB,aAAa,KAAK,aAAa,MAAM,EAC9I,aAAa,CAAC,KAAM,CAAC,QAAQ,EAC7B,OAAO,CACR,CAAA;IACH,CAAC;CACF;AAED,MAAM,OAAO,gCAAiC,SAAQ,UAAU;IAC9D,IAAI,GAAG,kCAAkC,CAAA;IAEzC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,+BAA+B,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEvE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -0,0 +1,39 @@
|
|
|
1
|
+
import { ParserRule } from "../types.js";
|
|
2
|
+
import { BaseRuleVisitor, getTagName, getAttribute, getAttributeValue } from "./rule-utils.js";
|
|
3
|
+
class IframeHasTitleVisitor extends BaseRuleVisitor {
|
|
4
|
+
visitHTMLOpenTagNode(node) {
|
|
5
|
+
this.checkIframeElement(node);
|
|
6
|
+
super.visitHTMLOpenTagNode(node);
|
|
7
|
+
}
|
|
8
|
+
checkIframeElement(node) {
|
|
9
|
+
const tagName = getTagName(node);
|
|
10
|
+
if (tagName !== "iframe") {
|
|
11
|
+
return;
|
|
12
|
+
}
|
|
13
|
+
const ariaHiddenAttribute = getAttribute(node, "aria-hidden");
|
|
14
|
+
if (ariaHiddenAttribute) {
|
|
15
|
+
const ariaHiddenValue = getAttributeValue(ariaHiddenAttribute);
|
|
16
|
+
if (ariaHiddenValue === "true") {
|
|
17
|
+
return;
|
|
18
|
+
}
|
|
19
|
+
}
|
|
20
|
+
const attribute = getAttribute(node, "title");
|
|
21
|
+
if (!attribute) {
|
|
22
|
+
this.addOffense("`<iframe>` elements must have a `title` attribute that describes the content of the frame for screen reader users.", node.location, "error");
|
|
23
|
+
return;
|
|
24
|
+
}
|
|
25
|
+
const value = getAttributeValue(attribute);
|
|
26
|
+
if (!value || value.trim() === "") {
|
|
27
|
+
this.addOffense("`<iframe>` elements must have a `title` attribute that describes the content of the frame for screen reader users.", node.location, "error");
|
|
28
|
+
}
|
|
29
|
+
}
|
|
30
|
+
}
|
|
31
|
+
export class HTMLIframeHasTitleRule extends ParserRule {
|
|
32
|
+
name = "html-iframe-has-title";
|
|
33
|
+
check(result, context) {
|
|
34
|
+
const visitor = new IframeHasTitleVisitor(this.name, context);
|
|
35
|
+
visitor.visit(result.value);
|
|
36
|
+
return visitor.offenses;
|
|
37
|
+
}
|
|
38
|
+
}
|
|
39
|
+
//# sourceMappingURL=html-iframe-has-title.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"html-iframe-has-title.js","sourceRoot":"","sources":["../../../src/rules/html-iframe-has-title.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,YAAY,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAA;AAK9F,MAAM,qBAAsB,SAAQ,eAAe;IACjD,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,kBAAkB,CAAC,IAAI,CAAC,CAAA;QAC7B,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;IAEO,kBAAkB,CAAC,IAAqB;QAC9C,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,CAAC,CAAA;QAEhC,IAAI,OAAO,KAAK,QAAQ,EAAE,CAAC;YACzB,OAAM;QACR,CAAC;QAED,MAAM,mBAAmB,GAAG,YAAY,CAAC,IAAI,EAAE,aAAa,CAAC,CAAA;QAC7D,IAAI,mBAAmB,EAAE,CAAC;YACxB,MAAM,eAAe,GAAG,iBAAiB,CAAC,mBAAmB,CAAC,CAAA;YAC9D,IAAI,eAAe,KAAK,MAAM,EAAE,CAAC;gBAC/B,OAAM;YACR,CAAC;QACH,CAAC;QAED,MAAM,SAAS,GAAG,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAE7C,IAAI,CAAC,SAAS,EAAE,CAAC;YACf,IAAI,CAAC,UAAU,CACb,oHAAoH,EACpH,IAAI,CAAC,QAAQ,EACb,OAAO,CACR,CAAA;YAED,OAAM;QACR,CAAC;QAED,MAAM,KAAK,GAAG,iBAAiB,CAAC,SAAS,CAAC,CAAA;QAE1C,IAAI,CAAC,KAAK,IAAI,KAAK,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;YAClC,IAAI,CAAC,UAAU,CACb,oHAAoH,EACpH,IAAI,CAAC,QAAQ,EACb,OAAO,CACR,CAAA;QACH,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,sBAAuB,SAAQ,UAAU;IACpD,IAAI,GAAG,uBAAuB,CAAA;IAE9B,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAE7D,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -5,10 +5,6 @@ class ImgRequireAltVisitor extends BaseRuleVisitor {
|
|
|
5
5
|
this.checkImgTag(node);
|
|
6
6
|
super.visitHTMLOpenTagNode(node);
|
|
7
7
|
}
|
|
8
|
-
visitHTMLSelfCloseTagNode(node) {
|
|
9
|
-
this.checkImgTag(node);
|
|
10
|
-
super.visitHTMLSelfCloseTagNode(node);
|
|
11
|
-
}
|
|
12
8
|
checkImgTag(node) {
|
|
13
9
|
const tagName = getTagName(node);
|
|
14
10
|
if (tagName !== "img") {
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-img-require-alt.js","sourceRoot":"","sources":["../../../src/rules/html-img-require-alt.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAA;AAE3E,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AAIxC,MAAM,oBAAqB,SAAQ,eAAe;IAChD,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,CAAA;QACtB,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;
|
|
1
|
+
{"version":3,"file":"html-img-require-alt.js","sourceRoot":"","sources":["../../../src/rules/html-img-require-alt.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAA;AAE3E,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AAIxC,MAAM,oBAAqB,SAAQ,eAAe;IAChD,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,CAAA;QACtB,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;IAEO,WAAW,CAAC,IAAqB;QACvC,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,CAAC,CAAA;QAEhC,IAAI,OAAO,KAAK,KAAK,EAAE,CAAC;YACtB,OAAM;QACR,CAAC;QAED,IAAI,CAAC,YAAY,CAAC,IAAI,EAAE,KAAK,CAAC,EAAE,CAAC;YAC/B,IAAI,CAAC,UAAU,CACb,oIAAoI,EACpI,IAAI,CAAC,QAAS,CAAC,QAAQ,EACvB,OAAO,CACR,CAAA;QACH,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,qBAAsB,SAAQ,UAAU;IACnD,IAAI,GAAG,sBAAsB,CAAA;IAE7B,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,oBAAoB,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAC5D,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAC3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -0,0 +1,43 @@
|
|
|
1
|
+
import { ParserRule } from "../types.js";
|
|
2
|
+
import { BaseRuleVisitor, getTagName, hasAttribute, getAttributeValue, findAttributeByName, getAttributes } from "./rule-utils.js";
|
|
3
|
+
class NavigationHasLabelVisitor extends BaseRuleVisitor {
|
|
4
|
+
visitHTMLOpenTagNode(node) {
|
|
5
|
+
this.checkNavigationElement(node);
|
|
6
|
+
super.visitHTMLOpenTagNode(node);
|
|
7
|
+
}
|
|
8
|
+
checkNavigationElement(node) {
|
|
9
|
+
const tagName = getTagName(node);
|
|
10
|
+
const isNavElement = tagName === "nav";
|
|
11
|
+
const hasNavigationRole = this.hasRoleNavigation(node);
|
|
12
|
+
if (!isNavElement && !hasNavigationRole) {
|
|
13
|
+
return;
|
|
14
|
+
}
|
|
15
|
+
const hasAriaLabel = hasAttribute(node, "aria-label");
|
|
16
|
+
const hasAriaLabelledby = hasAttribute(node, "aria-labelledby");
|
|
17
|
+
if (!hasAriaLabel && !hasAriaLabelledby) {
|
|
18
|
+
let message = `The navigation landmark should have a unique accessible name via \`aria-label\` or \`aria-labelledby\`. Remember that the name does not need to include "navigation" or "nav" since it will already be announced.`;
|
|
19
|
+
if (hasNavigationRole && !isNavElement) {
|
|
20
|
+
message += ` Additionally, you can safely drop the \`role="navigation"\` and replace it with the native HTML \`<nav>\` element.`;
|
|
21
|
+
}
|
|
22
|
+
this.addOffense(message, node.tag_name.location, "error");
|
|
23
|
+
}
|
|
24
|
+
}
|
|
25
|
+
hasRoleNavigation(node) {
|
|
26
|
+
const attributes = getAttributes(node);
|
|
27
|
+
const roleAttribute = findAttributeByName(attributes, "role");
|
|
28
|
+
if (!roleAttribute) {
|
|
29
|
+
return false;
|
|
30
|
+
}
|
|
31
|
+
const roleValue = getAttributeValue(roleAttribute);
|
|
32
|
+
return roleValue === "navigation";
|
|
33
|
+
}
|
|
34
|
+
}
|
|
35
|
+
export class HTMLNavigationHasLabelRule extends ParserRule {
|
|
36
|
+
name = "html-navigation-has-label";
|
|
37
|
+
check(result, context) {
|
|
38
|
+
const visitor = new NavigationHasLabelVisitor(this.name, context);
|
|
39
|
+
visitor.visit(result.value);
|
|
40
|
+
return visitor.offenses;
|
|
41
|
+
}
|
|
42
|
+
}
|
|
43
|
+
//# sourceMappingURL=html-navigation-has-label.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"html-navigation-has-label.js","sourceRoot":"","sources":["../../../src/rules/html-navigation-has-label.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,YAAY,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAA;AAKlI,MAAM,yBAA0B,SAAQ,eAAe;IACrD,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,CAAA;QACjC,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;IAEO,sBAAsB,CAAC,IAAqB;QAClD,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,CAAC,CAAA;QAChC,MAAM,YAAY,GAAG,OAAO,KAAK,KAAK,CAAA;QACtC,MAAM,iBAAiB,GAAG,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAA;QAEtD,IAAI,CAAC,YAAY,IAAI,CAAC,iBAAiB,EAAE,CAAC;YACxC,OAAM;QACR,CAAC;QAED,MAAM,YAAY,GAAG,YAAY,CAAC,IAAI,EAAE,YAAY,CAAC,CAAA;QACrD,MAAM,iBAAiB,GAAG,YAAY,CAAC,IAAI,EAAE,iBAAiB,CAAC,CAAA;QAE/D,IAAI,CAAC,YAAY,IAAI,CAAC,iBAAiB,EAAE,CAAC;YACxC,IAAI,OAAO,GAAG,mNAAmN,CAAA;YAEjO,IAAI,iBAAiB,IAAI,CAAC,YAAY,EAAE,CAAC;gBACvC,OAAO,IAAI,qHAAqH,CAAA;YAClI,CAAC;YAED,IAAI,CAAC,UAAU,CACb,OAAO,EACP,IAAI,CAAC,QAAS,CAAC,QAAQ,EACvB,OAAO,CACR,CAAA;QACH,CAAC;IACH,CAAC;IAEO,iBAAiB,CAAC,IAAqB;QAC7C,MAAM,UAAU,GAAG,aAAa,CAAC,IAAI,CAAC,CAAA;QACtC,MAAM,aAAa,GAAG,mBAAmB,CAAC,UAAU,EAAE,MAAM,CAAC,CAAA;QAE7D,IAAI,CAAC,aAAa,EAAE,CAAC;YACnB,OAAO,KAAK,CAAA;QACd,CAAC;QAED,MAAM,SAAS,GAAG,iBAAiB,CAAC,aAAa,CAAC,CAAA;QAElD,OAAO,SAAS,KAAK,YAAY,CAAA;IACnC,CAAC;CACF;AAED,MAAM,OAAO,0BAA2B,SAAQ,UAAU;IACxD,IAAI,GAAG,2BAA2B,CAAA;IAElC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,yBAAyB,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEjE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -0,0 +1,67 @@
|
|
|
1
|
+
import { ParserRule } from "../types.js";
|
|
2
|
+
import { BaseRuleVisitor, getTagName, hasAttribute, getAttributeValue, findAttributeByName, getAttributes } from "./rule-utils.js";
|
|
3
|
+
const INTERACTIVE_ELEMENTS = new Set([
|
|
4
|
+
"button", "summary", "input", "select", "textarea", "a"
|
|
5
|
+
]);
|
|
6
|
+
class NoAriaHiddenOnFocusableVisitor extends BaseRuleVisitor {
|
|
7
|
+
visitHTMLOpenTagNode(node) {
|
|
8
|
+
this.checkAriaHiddenOnFocusable(node);
|
|
9
|
+
super.visitHTMLOpenTagNode(node);
|
|
10
|
+
}
|
|
11
|
+
checkAriaHiddenOnFocusable(node) {
|
|
12
|
+
if (!this.hasAriaHiddenTrue(node))
|
|
13
|
+
return;
|
|
14
|
+
if (this.isFocusable(node)) {
|
|
15
|
+
this.addOffense(`Elements that are focusable should not have \`aria-hidden="true"\` because it will cause confusion for assistive technology users.`, node.tag_name.location, "error");
|
|
16
|
+
}
|
|
17
|
+
}
|
|
18
|
+
hasAriaHiddenTrue(node) {
|
|
19
|
+
const attributes = getAttributes(node);
|
|
20
|
+
const ariaHiddenAttr = findAttributeByName(attributes, "aria-hidden");
|
|
21
|
+
if (!ariaHiddenAttr)
|
|
22
|
+
return false;
|
|
23
|
+
const value = getAttributeValue(ariaHiddenAttr);
|
|
24
|
+
return value === "true";
|
|
25
|
+
}
|
|
26
|
+
isFocusable(node) {
|
|
27
|
+
const tagName = getTagName(node);
|
|
28
|
+
if (!tagName)
|
|
29
|
+
return false;
|
|
30
|
+
const tabIndexValue = this.getTabIndexValue(node);
|
|
31
|
+
if (tagName === "a") {
|
|
32
|
+
const hasHref = hasAttribute(node, "href");
|
|
33
|
+
if (!hasHref) {
|
|
34
|
+
return tabIndexValue !== null && tabIndexValue >= 0;
|
|
35
|
+
}
|
|
36
|
+
return tabIndexValue === null || tabIndexValue >= 0;
|
|
37
|
+
}
|
|
38
|
+
if (INTERACTIVE_ELEMENTS.has(tagName)) {
|
|
39
|
+
// Interactive elements are focusable unless tabindex is negative
|
|
40
|
+
return tabIndexValue === null || tabIndexValue >= 0;
|
|
41
|
+
}
|
|
42
|
+
else {
|
|
43
|
+
// Non-interactive elements are focusable only if tabindex >= 0
|
|
44
|
+
return tabIndexValue !== null && tabIndexValue >= 0;
|
|
45
|
+
}
|
|
46
|
+
}
|
|
47
|
+
getTabIndexValue(node) {
|
|
48
|
+
const attributes = getAttributes(node);
|
|
49
|
+
const tabIndexAttribute = findAttributeByName(attributes, "tabindex");
|
|
50
|
+
if (!tabIndexAttribute)
|
|
51
|
+
return null;
|
|
52
|
+
const value = getAttributeValue(tabIndexAttribute);
|
|
53
|
+
if (!value)
|
|
54
|
+
return null;
|
|
55
|
+
const parsed = parseInt(value, 10);
|
|
56
|
+
return isNaN(parsed) ? null : parsed;
|
|
57
|
+
}
|
|
58
|
+
}
|
|
59
|
+
export class HTMLNoAriaHiddenOnFocusableRule extends ParserRule {
|
|
60
|
+
name = "html-no-aria-hidden-on-focusable";
|
|
61
|
+
check(result, context) {
|
|
62
|
+
const visitor = new NoAriaHiddenOnFocusableVisitor(this.name, context);
|
|
63
|
+
visitor.visit(result.value);
|
|
64
|
+
return visitor.offenses;
|
|
65
|
+
}
|
|
66
|
+
}
|
|
67
|
+
//# sourceMappingURL=html-no-aria-hidden-on-focusable.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"html-no-aria-hidden-on-focusable.js","sourceRoot":"","sources":["../../../src/rules/html-no-aria-hidden-on-focusable.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,YAAY,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAA;AAKlI,MAAM,oBAAoB,GAAG,IAAI,GAAG,CAAC;IACnC,QAAQ,EAAE,SAAS,EAAE,OAAO,EAAE,QAAQ,EAAE,UAAU,EAAE,GAAG;CACxD,CAAC,CAAA;AAEF,MAAM,8BAA+B,SAAQ,eAAe;IAC1D,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,0BAA0B,CAAC,IAAI,CAAC,CAAA;QACrC,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;IAEO,0BAA0B,CAAC,IAAqB;QACtD,IAAI,CAAC,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC;YAAE,OAAM;QAEzC,IAAI,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,EAAE,CAAC;YAC3B,IAAI,CAAC,UAAU,CACb,oIAAoI,EACpI,IAAI,CAAC,QAAS,CAAC,QAAQ,EACvB,OAAO,CACR,CAAA;QACH,CAAC;IACH,CAAC;IAEO,iBAAiB,CAAC,IAAqB;QAC7C,MAAM,UAAU,GAAG,aAAa,CAAC,IAAI,CAAC,CAAA;QACtC,MAAM,cAAc,GAAG,mBAAmB,CAAC,UAAU,EAAE,aAAa,CAAC,CAAA;QAErE,IAAI,CAAC,cAAc;YAAE,OAAO,KAAK,CAAA;QAEjC,MAAM,KAAK,GAAG,iBAAiB,CAAC,cAAc,CAAC,CAAA;QAE/C,OAAO,KAAK,KAAK,MAAM,CAAA;IACzB,CAAC;IAEO,WAAW,CAAC,IAAqB;QACvC,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,CAAC,CAAA;QAChC,IAAI,CAAC,OAAO;YAAE,OAAO,KAAK,CAAA;QAE1B,MAAM,aAAa,GAAG,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAA;QAEjD,IAAI,OAAO,KAAK,GAAG,EAAE,CAAC;YACpB,MAAM,OAAO,GAAG,YAAY,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;YAE1C,IAAI,CAAC,OAAO,EAAE,CAAC;gBACb,OAAO,aAAa,KAAK,IAAI,IAAI,aAAa,IAAI,CAAC,CAAA;YACrD,CAAC;YAED,OAAO,aAAa,KAAK,IAAI,IAAI,aAAa,IAAI,CAAC,CAAA;QACrD,CAAC;QAED,IAAI,oBAAoB,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;YACtC,iEAAiE;YACjE,OAAO,aAAa,KAAK,IAAI,IAAI,aAAa,IAAI,CAAC,CAAA;QACrD,CAAC;aAAM,CAAC;YACN,+DAA+D;YAC/D,OAAO,aAAa,KAAK,IAAI,IAAI,aAAa,IAAI,CAAC,CAAA;QACrD,CAAC;IACH,CAAC;IAEO,gBAAgB,CAAC,IAAqB;QAC5C,MAAM,UAAU,GAAG,aAAa,CAAC,IAAI,CAAC,CAAA;QACtC,MAAM,iBAAiB,GAAG,mBAAmB,CAAC,UAAU,EAAE,UAAU,CAAC,CAAA;QAErE,IAAI,CAAC,iBAAiB;YAAE,OAAO,IAAI,CAAA;QAEnC,MAAM,KAAK,GAAG,iBAAiB,CAAC,iBAAiB,CAAC,CAAA;QAClD,IAAI,CAAC,KAAK;YAAE,OAAO,IAAI,CAAA;QAEvB,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,EAAE,EAAE,CAAC,CAAA;QAElC,OAAO,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAA;IACtC,CAAC;CACF;AAED,MAAM,OAAO,+BAAgC,SAAQ,UAAU;IAC7D,IAAI,GAAG,kCAAkC,CAAA;IAEzC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,8BAA8B,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEtE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -1,33 +1,30 @@
|
|
|
1
|
-
import { BaseRuleVisitor, forEachAttribute } from "./rule-utils.js";
|
|
2
1
|
import { ParserRule } from "../types.js";
|
|
3
|
-
|
|
2
|
+
import { AttributeVisitorMixin } from "./rule-utils.js";
|
|
3
|
+
class NoDuplicateAttributesVisitor extends AttributeVisitorMixin {
|
|
4
|
+
attributeNames = new Map();
|
|
4
5
|
visitHTMLOpenTagNode(node) {
|
|
5
|
-
this.
|
|
6
|
+
this.attributeNames.clear();
|
|
6
7
|
super.visitHTMLOpenTagNode(node);
|
|
8
|
+
this.reportDuplicates();
|
|
7
9
|
}
|
|
8
|
-
|
|
9
|
-
this.
|
|
10
|
-
super.visitHTMLSelfCloseTagNode(node);
|
|
10
|
+
checkStaticAttributeStaticValue({ attributeName, attributeNode }) {
|
|
11
|
+
this.trackAttributeName(attributeName, attributeNode);
|
|
11
12
|
}
|
|
12
|
-
|
|
13
|
-
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
if (nameNodes.length > 1) {
|
|
28
|
-
for (let i = 1; i < nameNodes.length; i++) {
|
|
29
|
-
const nameNode = nameNodes[i];
|
|
30
|
-
this.addOffense(`Duplicate attribute \`${attributeName}\` found on tag. Remove the duplicate occurrence.`, nameNode.location, "error");
|
|
13
|
+
checkStaticAttributeDynamicValue({ attributeName, attributeNode }) {
|
|
14
|
+
this.trackAttributeName(attributeName, attributeNode);
|
|
15
|
+
}
|
|
16
|
+
trackAttributeName(attributeName, attributeNode) {
|
|
17
|
+
if (!this.attributeNames.has(attributeName)) {
|
|
18
|
+
this.attributeNames.set(attributeName, []);
|
|
19
|
+
}
|
|
20
|
+
this.attributeNames.get(attributeName).push(attributeNode);
|
|
21
|
+
}
|
|
22
|
+
reportDuplicates() {
|
|
23
|
+
for (const [attributeName, attributeNodes] of this.attributeNames) {
|
|
24
|
+
if (attributeNodes.length > 1) {
|
|
25
|
+
for (let i = 1; i < attributeNodes.length; i++) {
|
|
26
|
+
const attributeNode = attributeNodes[i];
|
|
27
|
+
this.addOffense(`Duplicate attribute \`${attributeName}\` found on tag. Remove the duplicate occurrence.`, attributeNode.name.location, "error");
|
|
31
28
|
}
|
|
32
29
|
}
|
|
33
30
|
}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-no-duplicate-attributes.js","sourceRoot":"","sources":["../../../src/rules/html-no-duplicate-attributes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-no-duplicate-attributes.js","sourceRoot":"","sources":["../../../src/rules/html-no-duplicate-attributes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAA;AACxC,OAAO,EAAE,qBAAqB,EAAuE,MAAM,iBAAiB,CAAA;AAK5H,MAAM,4BAA6B,SAAQ,qBAAqB;IACtD,cAAc,GAAG,IAAI,GAAG,EAA+B,CAAA;IAE/D,oBAAoB,CAAC,IAAqB;QACxC,IAAI,CAAC,cAAc,CAAC,KAAK,EAAE,CAAA;QAC3B,KAAK,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAA;QAChC,IAAI,CAAC,gBAAgB,EAAE,CAAA;IACzB,CAAC;IAGS,+BAA+B,CAAC,EAAE,aAAa,EAAE,aAAa,EAAoC;QAC1G,IAAI,CAAC,kBAAkB,CAAC,aAAa,EAAE,aAAa,CAAC,CAAA;IACvD,CAAC;IAES,gCAAgC,CAAC,EAAE,aAAa,EAAE,aAAa,EAAqC;QAC5G,IAAI,CAAC,kBAAkB,CAAC,aAAa,EAAE,aAAa,CAAC,CAAA;IACvD,CAAC;IAEO,kBAAkB,CAAC,aAAqB,EAAE,aAAgC;QAChF,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,aAAa,CAAC,EAAE,CAAC;YAC5C,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,aAAa,EAAE,EAAE,CAAC,CAAA;QAC5C,CAAC;QAED,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,aAAa,CAAE,CAAC,IAAI,CAAC,aAAa,CAAC,CAAA;IAC7D,CAAC;IAEO,gBAAgB;QACtB,KAAK,MAAM,CAAC,aAAa,EAAE,cAAc,CAAC,IAAI,IAAI,CAAC,cAAc,EAAE,CAAC;YAClE,IAAI,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBAC9B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,cAAc,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;oBAC/C,MAAM,aAAa,GAAG,cAAc,CAAC,CAAC,CAAC,CAAA;oBAEvC,IAAI,CAAC,UAAU,CACb,yBAAyB,aAAa,mDAAmD,EACzF,aAAa,CAAC,IAAK,CAAC,QAAQ,EAC5B,OAAO,CACR,CAAA;gBACH,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;CACF;AAED,MAAM,OAAO,6BAA8B,SAAQ,UAAU;IAC3D,IAAI,GAAG,8BAA8B,CAAA;IAErC,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,4BAA4B,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAEpE,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -1,8 +1,8 @@
|
|
|
1
|
-
import { AttributeVisitorMixin } from "./rule-utils";
|
|
2
1
|
import { ParserRule } from "../types";
|
|
2
|
+
import { AttributeVisitorMixin } from "./rule-utils";
|
|
3
3
|
class NoDuplicateIdsVisitor extends AttributeVisitorMixin {
|
|
4
4
|
documentIds = new Set();
|
|
5
|
-
|
|
5
|
+
checkStaticAttributeStaticValue({ attributeName, attributeValue, attributeNode }) {
|
|
6
6
|
if (attributeName.toLowerCase() !== "id")
|
|
7
7
|
return;
|
|
8
8
|
if (!attributeValue)
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"html-no-duplicate-ids.js","sourceRoot":"","sources":["../../../src/rules/html-no-duplicate-ids.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,
|
|
1
|
+
{"version":3,"file":"html-no-duplicate-ids.js","sourceRoot":"","sources":["../../../src/rules/html-no-duplicate-ids.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,UAAU,CAAA;AACrC,OAAO,EAAE,qBAAqB,EAAoC,MAAM,cAAc,CAAA;AAKtF,MAAM,qBAAsB,SAAQ,qBAAqB;IAC/C,WAAW,GAAgB,IAAI,GAAG,EAAU,CAAA;IAE1C,+BAA+B,CAAC,EAAE,aAAa,EAAE,cAAc,EAAE,aAAa,EAAoC;QAC1H,IAAI,aAAa,CAAC,WAAW,EAAE,KAAK,IAAI;YAAE,OAAM;QAChD,IAAI,CAAC,cAAc;YAAE,OAAM;QAE3B,MAAM,EAAE,GAAG,cAAc,CAAC,IAAI,EAAE,CAAA;QAEhC,IAAI,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC;YAC7B,IAAI,CAAC,UAAU,CACb,kBAAkB,EAAE,iDAAiD,EACrE,aAAa,CAAC,QAAQ,EACtB,OAAO,CACR,CAAA;YAED,OAAM;QACR,CAAC;QAED,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,EAAE,CAAC,CAAA;IAC1B,CAAC;CACF;AAED,MAAM,OAAO,sBAAuB,SAAQ,UAAU;IACpD,IAAI,GAAG,uBAAuB,CAAA;IAE9B,KAAK,CAAC,MAAmB,EAAE,OAA8B;QACvD,MAAM,OAAO,GAAG,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,EAAE,OAAO,CAAC,CAAA;QAE7D,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;QAE3B,OAAO,OAAO,CAAC,QAAQ,CAAA;IACzB,CAAC;CACF"}
|
|
@@ -5,10 +5,6 @@ class NoEmptyHeadingsVisitor extends BaseRuleVisitor {
|
|
|
5
5
|
this.checkHeadingElement(node);
|
|
6
6
|
super.visitHTMLElementNode(node);
|
|
7
7
|
}
|
|
8
|
-
visitHTMLSelfCloseTagNode(node) {
|
|
9
|
-
this.checkSelfClosingHeading(node);
|
|
10
|
-
super.visitHTMLSelfCloseTagNode(node);
|
|
11
|
-
}
|
|
12
8
|
checkHeadingElement(node) {
|
|
13
9
|
if (!node.open_tag || node.open_tag.type !== "AST_HTML_OPEN_TAG_NODE") {
|
|
14
10
|
return;
|
|
@@ -30,23 +26,6 @@ class NoEmptyHeadingsVisitor extends BaseRuleVisitor {
|
|
|
30
26
|
this.addOffense(`Heading element ${elementDescription} must not be empty. Provide accessible text content for screen readers and SEO.`, node.location, "error");
|
|
31
27
|
}
|
|
32
28
|
}
|
|
33
|
-
checkSelfClosingHeading(node) {
|
|
34
|
-
const tagName = getTagName(node);
|
|
35
|
-
if (!tagName) {
|
|
36
|
-
return;
|
|
37
|
-
}
|
|
38
|
-
// Check if it's a standard heading tag (h1-h6) or has role="heading"
|
|
39
|
-
const isStandardHeading = HEADING_TAGS.has(tagName);
|
|
40
|
-
const isAriaHeading = this.hasHeadingRole(node);
|
|
41
|
-
if (!isStandardHeading && !isAriaHeading) {
|
|
42
|
-
return;
|
|
43
|
-
}
|
|
44
|
-
// Self-closing headings are always empty
|
|
45
|
-
const elementDescription = isStandardHeading
|
|
46
|
-
? `\`<${tagName}>\``
|
|
47
|
-
: `\`<${tagName} role="heading">\``;
|
|
48
|
-
this.addOffense(`Heading element ${elementDescription} must not be empty. Provide accessible text content for screen readers and SEO.`, node.tag_name.location, "error");
|
|
49
|
-
}
|
|
50
29
|
isEmptyHeading(node) {
|
|
51
30
|
if (!node.body || node.body.length === 0) {
|
|
52
31
|
return true;
|