[#18671], Fix a11y issue for IE platform (#19037)

* [#18671], Fix a11y issue for IE platform * [#18671]: PR Feedback fix Used tryFocus instead of element.focus * [#18671]: Update implementation approach * [#18671], Remove unused import
ampproject · Oct 30, 2018 · dd80243 · dd80243
1 parent 36fd394
commit dd80243
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/src/service/navigation.js b/src/service/navigation.js
@@ -20,6 +20,7 @@ import {
   escapeCssSelectorIdent,
   isIframed,
   openWindowDialog,
+  tryFocus,
 } from '../dom';
 import {dev, user} from '../log';
 import {dict} from '../utils/object';
@@ -458,6 +459,19 @@ export class Navigation {
         if (targetAttr != '_top' && targetAttr != '_blank') {
           target.setAttribute('target', '_blank');
         }
+        return; // bail early.
+      }
+
+      // Accessibility fix for IE browser
+      if (Services.platformFor(this.ampdoc.win).isIe()) {
+        const internalTargetElmId = tgtLoc.hash.substring(1);
+        const internalElm = this.ampdoc.getElementById(internalTargetElmId);
+        if (internalElm) {
+          if (!(/^(?:a|select|input|button|textarea)$/i.test(internalElm.tagName))) {
+            internalElm.tabIndex = -1;
+          }
+          tryFocus(internalElm);
+        }
       }
       return;
     }