Last modified: 2013-09-15 22:56:38 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T56015, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 54015 - Iceweasel omits "shift" from shortcut tooltips for "View history" etc. tabs
Iceweasel omits "shift" from shortcut tooltips for "View history" etc. tabs
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.22.0
PC Linux
: Lowest minor (vote)
: 1.22.0 release
Assigned To: Bartosz Dziewoński
http://en.wikipedia.org/
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-11 10:15 UTC by Nirakka
Modified: 2013-09-15 22:56 UTC (History)
5 users (show)

See Also:
Web browser: Other
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Nirakka 2013-09-11 10:15:07 UTC
The tooltips (on mouseover) for MediaWiki shortcuts omit the necessary shift key. For instance, the history button claims shortcut [alt-h] (instead of [alt-shift-h]). This issue occurs in Iceweasel 20.0 on Debian Wheezy.
Comment 1 Andre Klapper 2013-09-11 14:51:53 UTC
With Firefox 23.0.1 on Fedora19 on en.wikipedia.org, I get the correct keyboard shortcuts when hovering the mouse over "Read | Edit source | Edit beta | View history".

The HTML source code of the page says
<a accesskey="h" title="Past versions of this page [alt-shift-h]" href="/w/index.php?title=Berlin&action=history">

What makes you think that this is a Wikimedia and not a Iceweasel problem?
Comment 2 Nirakka 2013-09-11 15:01:02 UTC
The problem does not occur on other MediaWiki installations, e. g. http://wiki.bnc4free.com/index.php/Main_Page shows the correct shortcuts in Iceweasel.
Comment 3 Bartosz Dziewoński 2013-09-11 15:29:47 UTC
That site is running an older version of MediaWiki (1.18).

The shortcuts shown in the tooltips are browser-dependent. "alt-shift" is meant for Firefox 2.x and later, "alt" is the default when browser can't be detected. 

It seems like something makes Seamonkey 20 undetectable for MediaWiki 1.22 alpha (the version Wikipedias run).
Comment 4 Michael M. 2013-09-12 08:04:33 UTC
Well, jQuery.client correctly detects Iceweasel as iceweasel [1] (in 1.18 Iceweasel was detected as firefox), but mediawiki.util doesn't have any rule for iceweasel (only for firefox [2]), so the displayed prefix falls back to "alt-".

Alternativly, fix bug 37239 to reduce the mess that's necessary to find the correct prefix.

[1] https://git.wikimedia.org/blob/mediawiki%2Fcore.git/554a5d04433207c6386d8c54281ac4e58b220a47/tests%2Fqunit%2Fsuites%2Fresources%2Fjquery%2Fjquery.client.test.js#L193
[2] https://git.wikimedia.org/blob/mediawiki%2Fcore.git/554a5d04433207c6386d8c54281ac4e58b220a47/resources%2Fmediawiki%2Fmediawiki.util.js#L56
Comment 5 Bartosz Dziewoński 2013-09-15 16:39:59 UTC
Hm, okay, that sounds simple enough – thanks for the analysis, Michael. You should get [[mw:developer access]] and start fixing those bugs yourself :)

Using the accessKeyLabel property per bug 37239 would be a little annoying to implement (particularly because Firefox and friends, still the only browsers supporting it, use a different format than MediaWiki – Alt+Shift+X instead of alt-shift-x) and not really worth it (because, again, no other browsers support this).

I'll submit a patch in a second – I'd appreciate if you could test it.
Comment 6 Gerrit Notification Bot 2013-09-15 16:43:05 UTC
Change 84225 had a related patch set uploaded by Bartosz Dziewoński:
mediawiki.util: Detect Iceweasel for accesskeys

https://gerrit.wikimedia.org/r/84225
Comment 7 Gerrit Notification Bot 2013-09-15 19:41:02 UTC
Change 84225 merged by jenkins-bot:
mediawiki.util: Detect Iceweasel for accesskeys

https://gerrit.wikimedia.org/r/84225
Comment 8 Bartosz Dziewoński 2013-09-15 19:41:58 UTC
Marking as fixed, it'd be nice if one of you could verify that the patch does indeed fix the issue :)
Comment 9 Nirakka 2013-09-15 22:18:18 UTC
If the change should be visible immediately, the patch doesn't work. I still get [alt-h] for history on English wikipedia. :/
Comment 10 Bartosz Dziewoński 2013-09-15 22:56:38 UTC
No, it will only get deployed to Wikipedias and other sites on September 26. The deployment calendar is available at https://www.mediawiki.org/wiki/MediaWiki_1.22/Roadmap

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links