Skip to content

Commit

Permalink
fix(aria-allowed-role): allow role=presentation on hr (#2300)
Browse files Browse the repository at this point in the history
* fix(aria-allowed-role): allow role=presentation on hr

* add test
  • Loading branch information
straker authored Jun 18, 2020
1 parent 9857978 commit b524ea9
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
2 changes: 2 additions & 0 deletions lib/commons/aria/lookup-table.js
Original file line number Diff line number Diff line change
Expand Up @@ -1429,6 +1429,7 @@ lookupTable.role = {
'h5',
'h6',
'header',
'hr',
'iframe',
'li',
'ol',
Expand Down Expand Up @@ -1515,6 +1516,7 @@ lookupTable.role = {
'h5',
'h6',
'header',
'hr',
'iframe',
'li',
'ol',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -167,6 +167,8 @@ <h1 id="pass-h1-valid-role" role="none"></h1>
<ul>
<li id="pass-dpub-7" role="doc-endnote">ok</li>
</ul>
<hr id="hr-presentation" role="presentation" />
<hr id="hr-none" role="none" />

<dd id="fail-dd-no-role" role="link"></dd>
<dt id="fail-dt-no-role" role="banner"></dt>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,9 @@
["#pass-dpub-4"],
["#pass-dpub-5"],
["#pass-dpub-6"],
["#pass-dpub-7"]
["#pass-dpub-7"],
["#hr-presentation"],
["#hr-none"]
],
"violations": [
["#fail-dd-no-role"],
Expand Down

0 comments on commit b524ea9

Please sign in to comment.