Skip to content

Commit

Permalink
fix(820): check for all labelable elements
Browse files Browse the repository at this point in the history
  • Loading branch information
skippednote committed Jun 26, 2020
1 parent c623b33 commit 81ce352
Show file tree
Hide file tree
Showing 3 changed files with 44 additions and 32 deletions.
13 changes: 12 additions & 1 deletion src/compiler/compile/nodes/Element.ts
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,17 @@ const a11y_no_onchange = new Set([
'option'
]);

const a11y_labelable = new Set([
"button",
"input",
"keygen",
"meter",
"output",
"progress",
"select",
"textarea"
]);

const invisible_elements = new Set(['meta', 'html', 'script', 'style']);

const valid_modifiers = new Set([
Expand Down Expand Up @@ -508,7 +519,7 @@ export default class Element extends Node {
}

if (this.name === 'label') {
const has_input_child = this.children.some(i => (i instanceof Element && i.name === 'input' ));
const has_input_child = this.children.some(i => (i instanceof Element && a11y_labelable.has(i.name) ));
if (!attribute_map.has('for') && !has_input_child) {
component.warn(this, {
code: `a11y-label-has-associated-control`,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,5 @@
<label for="id">B</label>

<label>C <input type="text" /></label>
<label>D <span></span></label>
<label>D <button>D</button></label>
<label>E <span></span></label>
Original file line number Diff line number Diff line change
@@ -1,32 +1,32 @@
[
{
"code": "a11y-label-has-associated-control",
"end": {
"character": 16,
"column": 16,
"line": 1
},
"message": "A11y: A form label must be associated with a control.",
"pos": 0,
"start": {
"character": 0,
"column": 0,
"line": 1
}
},
{
"code": "a11y-label-has-associated-control",
"end": {
"character": 113,
"column": 30,
"line": 5
},
"message": "A11y: A form label must be associated with a control.",
"pos": 83,
"start": {
"character": 83,
"column": 0,
"line": 5
}
}
{
"code": "a11y-label-has-associated-control",
"end": {
"character": 16,
"column": 16,
"line": 1
},
"message": "A11y: A form label must be associated with a control.",
"pos": 0,
"start": {
"character": 0,
"column": 0,
"line": 1
}
},
{
"code": "a11y-label-has-associated-control",
"end": {
"character": 149,
"column": 30,
"line": 6
},
"message": "A11y: A form label must be associated with a control.",
"pos": 119,
"start": {
"character": 119,
"column": 0,
"line": 6
}
}
]

0 comments on commit 81ce352

Please sign in to comment.