Focus active navigation item on popup. Fix #1049.

FelisCatus · Apr 13, 2017 · 3854a01 · 3854a01
1 parent 118c358
commit 3854a01
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/omega-web/src/popup/js/keyboard.js b/omega-web/src/popup/js/keyboard.js
@@ -48,6 +48,11 @@
         handler();
       }
     });
+
+    $script.ready('om-profile-items', function() {
+      var activeNavLink = document.querySelector('.om-nav-item.om-active > a');
+      if (activeNavLink) activeNavLink.focus();
+    });
   }
 
   function tabbableElementsOnly(node) {

diff --git a/omega-web/src/popup/js/profiles.js b/omega-web/src/popup/js/profiles.js
@@ -5,7 +5,7 @@
 
   var profileTemplate = document.getElementById('js-profile-tpl')
     .cloneNode(true);
-  profileTemplate.classList.remove('om-profile-tpl');
+  profileTemplate.removeAttribute('id');
 
   var iconForProfileType = {
     'DirectProfile': 'glyphicon-transfer',
@@ -34,6 +34,7 @@
       return;
     }
     addProfilesItems(state);
+    $script.done('om-profile-items');
     updateOtherItems(state);
   }