Merge "A11y - Keyboard focus moves only to interactive elements on Dashboard"

polygerrit-ui/app/elements/change-list/gr-change-list-item/gr-change-list-item_html.js

@@ -45,13 +45,6 @@ export const htmlTemplate = html`
       white-space: nowrap;
       width: 100%;
     }
-    .content a {
-      display: block;
-      overflow: hidden;
-      text-overflow: ellipsis;
-      white-space: nowrap;
-      width: 100%;
-    }
     .comments,
     .reviewers {
       white-space: nowrap;

polygerrit-ui/app/elements/change-list/gr-change-list/gr-change-list.js

@@ -168,7 +168,6 @@ class GrChangeList extends mixinBehaviors( [
   /** @override */
   ready() {
     super.ready();
-    this._ensureAttribute('tabindex', 0);
     this.$.restAPI.getConfig().then(config => {
       this._config = config;
     });
@@ -296,6 +295,11 @@ class GrChangeList extends mixinBehaviors( [
     return idx == selectedIndex;
   }
 
+  _computeTabIndex(sectionIndex, index, selectedIndex) {
+    return this._computeItemSelected(sectionIndex, index, selectedIndex)
+      ? 0 : undefined;
+  }
+
   _computeItemNeedsReview(account, change, showReviewedState) {
     return showReviewedState && !change.reviewed &&
         !change.work_in_progress &&

polygerrit-ui/app/elements/change-list/gr-change-list/gr-change-list_html.js

@@ -138,7 +138,7 @@ export const htmlTemplate = html`
             visible-change-table-columns="[[visibleChangeTableColumns]]"
             show-number="[[showNumber]]"
             show-star="[[showStar]]"
-            tabindex="0"
+            tabindex$="[[_computeTabIndex(sectionIndex, index, selectedIndex)]]"
             label-names="[[labelNames]]"
           ></gr-change-list-item>
         </template>