fix(rules): Allow focusable role=application elements

dequelabs · May 13, 2018 · b5de450 · b5de450
1 parent 5644b95
commit b5de450
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/lib/checks/aria/valid-scrollable-semantics.js b/lib/checks/aria/valid-scrollable-semantics.js
@@ -14,6 +14,7 @@ const VALID_TAG_NAMES_FOR_SCROLLABLE_REGIONS = {
  * appropriate for scrollable elements found in the focus order.
  */
 const VALID_ROLES_FOR_SCROLLABLE_REGIONS = {
+	application: true,
 	banner: false,
 	complementary: true,
 	contentinfo: true,

diff --git a/test/integration/rules/focus-order-semantics/focus-order-semantics.html b/test/integration/rules/focus-order-semantics/focus-order-semantics.html
@@ -28,6 +28,7 @@ <h4>Valid landmark roles for scrollable containers</h4>
 		<div id="pass5" role="contentinfo" tabindex="0"></div>
 		<div id="pass6" role="main" tabindex="0"></div>
 		<div id="pass7" role="region" tabindex="0"></div>
+		<div id="pass8" role="application" tabindex="0"></div>
 	</div>
 	<h4>Valid scrollable HTML tags for scrollable regions, not selected by this rule</h4>
 	<div>

diff --git a/test/integration/rules/focus-order-semantics/focus-order-semantics.json b/test/integration/rules/focus-order-semantics/focus-order-semantics.json
@@ -8,7 +8,8 @@
 		["#pass4"],
 		["#pass5"],
 		["#pass6"],
-		["#pass7"]
+		["#pass7"],
+		["#pass8"]
 	],
 	"violations": [
 		["#violation1"],