From 4203843b0fb18dd876fb203cc99d79cf99e09e6f Mon Sep 17 00:00:00 2001 From: petarpetkovic Date: Mon, 16 Oct 2017 15:42:03 +0200 Subject: [PATCH] Fix tabindex applied twice on RC top section - Remove tabindex = 0 applied to ButtonElement and leave only tabindex on ButtonElement-button, so keyboard focus does not catch one time too much. Bug: T175893 Change-Id: I5bf759dd2cdcb1a719f1ca7844253025813f8e9a --- .../ui/mw.rcfilters.ui.RcTopSectionWidget.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.RcTopSectionWidget.js b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.RcTopSectionWidget.js index ca76046c0c..2c99d3d2b6 100644 --- a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.RcTopSectionWidget.js +++ b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.RcTopSectionWidget.js @@ -48,7 +48,8 @@ widget.switchTopLinks( 'collapsed' ); } ); - this.$topLinks.find( '.mw-recentchanges-toplinks-title' ).replaceWith( toplinksTitle.$element ); + this.$topLinks.find( '.mw-recentchanges-toplinks-title' ) + .replaceWith( toplinksTitle.$element.removeAttr( 'tabIndex' ) ); // Create two positions for the toplinks to toggle between // in the table (first cell) or up above it -- 2.20.1