From: petarpetkovic Date: Mon, 16 Oct 2017 13:42:03 +0000 (+0200) Subject: Fix tabindex applied twice on RC top section X-Git-Tag: 1.31.0-rc.0~1760^2 X-Git-Url: http://git.cyclocoop.org/%7B%24www_url%7Dadmin/password.php?a=commitdiff_plain;h=4203843b0fb18dd876fb203cc99d79cf99e09e6f;p=lhc%2Fweb%2Fwiklou.git 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 --- 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