Bug 1247110 - Synced Tabs Sidebar - Fixed keyboard navigation. r=markh

browser/components/syncedtabs/SyncedTabsListStore.js

@@ -151,7 +151,7 @@ Object.assign(SyncedTabsListStore.prototype, EventEmitter.prototype, {
     if (parent <= -1) {
       parentRow = 0;
     } else if (parent >= maxParentRow) {
-      parentRow = maxParentRow - 1;
+      return;
     }
 
     let childRow = child;

browser/components/syncedtabs/TabListView.js

@@ -95,6 +95,9 @@ TabListView.prototype = {
         this._renderClient(client);
       }
     }
+    if (this.list.firstChild) {
+      this.list.firstChild.querySelector(".item.tab:first-child .item-title").setAttribute("tabindex", 2);
+    }
   },
 
   destroy() {

browser/components/syncedtabs/sidebar.xhtml

@@ -66,7 +66,7 @@
 
     <template id="tabs-container-template">
       <div class="tabs-container">
-        <div class="list" role="listbox" tabindex="1"></div>
+        <div class="list" role="listbox"></div>
       </div>
     </template>
 
@@ -97,7 +97,7 @@
         <div class="sidebar-search-container tabs-container sync-state">
           <div class="search-box compact">
             <div class="textbox-input-box">
-              <input type="text" class="tabsFilter textbox-input"/>
+              <input type="text" class="tabsFilter textbox-input" tabindex="1"/>
               <div class="textbox-search-icons">
                 <a class="textbox-search-clear"></a>
                 <a class="textbox-search-icon"></a>
@@ -107,7 +107,7 @@
         </div>
       </div>
       <!-- the scrollable content area where our templates are inserted -->
-      <div id="template-container" class="content-scrollable">
+      <div id="template-container" class="content-scrollable" tabindex="-1">
       </div>
     </div>
   </body>