Last modified: 2014-07-08 19:35:47 UTC
Bug 50047 and bug 51918 have regressed for Firefox – possibly as a result of Ed's changes to how the buttons work? Yay for Firefox and its attitude to tab indexes.
Looks like the same issue as bug 66824
So adding this.reviewButton.$element.attr( 'tabindex', '0' ).find( 'a' ).removeAttr( 'tabindex' ); fixed this... Wtf firefox?
Change 143055 had a related patch set uploaded by Alex Monk: Set tabindex on button <div> instead of <a> https://gerrit.wikimedia.org/r/143055
Change 143055 merged by jenkins-bot: Set tabindex on button <div> instead of <a> https://gerrit.wikimedia.org/r/143055
Well that's at least a workaround. I wonder if this might be an upstream bug...
*** Bug 67153 has been marked as a duplicate of this bug. ***