feat(is-focusable): account for negative tabindex

- add option arg that enables looking for negative tabindex (defaults to false) references: #3500
dequelabs · Aug 16, 2022 · 124c1a3 · 124c1a3
1 parent e4de11f
commit 124c1a3
Show file tree

Hide file tree

Showing 2 changed files with 111 additions and 82 deletions.
diff --git a/lib/commons/dom/is-focusable.js b/lib/commons/dom/is-focusable.js
@@ -9,9 +9,10 @@ import { getNodeFromTree } from '../../core/utils';
  * @memberof axe.commons.dom
  * @instance
  * @param {HTMLElement} el The HTMLElement
+ * @param {object} options Optional. If set to {programatically: false}, the function will return true
  * @return {Boolean} The element's focusability status
  */
-export default function isFocusable(el) {
+export default function isFocusable(el, options = { programmatically: false }) {
   const vNode = el instanceof AbstractVirtualNode ? el : getNodeFromTree(el);
 
   if (vNode.props.nodeType !== 1) {
@@ -23,8 +24,16 @@ export default function isFocusable(el) {
   } else if (isNativelyFocusable(vNode)) {
     return true;
   }
+
   // check if the tabindex is specified and a parseable number
   var tabindex = vNode.attr('tabindex');
+  if (
+    tabindex &&
+    options.programmatically === true &&
+    parseInt(tabindex, 10) === -1
+  ) {
+    return false;
+  }
   if (tabindex && !isNaN(parseInt(tabindex, 10))) {
     return true;
   }