Page MenuHomePhabricator

Language change tooltip is broken
Closed, ResolvedPublic


See screenshot ( on master:

pasted_file (193×411 px, 11 KB)

Same happens with other wikis at master.

Likely to be case of,

Event Timeline

Broken here means:

  • Missing the caret/triangle pointing to the element
  • Position of the tooltip

Like the same cause as in T160852#3112935 with the same fix. But this one is more urgent to fix, in my opinion.

Amire80 triaged this task as High priority.Mar 24 2017, 7:24 AM
Volker_E edited projects, added OOUI (OOjs-UI-0.20.1); removed OOUI.

The OOjs UI issue causing the exception is now fixed, but the popup is not being positioned correctly now, for the same reasons as T162101.

Change 346212 had a related patch set uploaded (by Bartosz Dziewoński):
[mediawiki/extensions/UniversalLanguageSelector@master] Fix language change tooltip positioning

Change 346212 merged by jenkins-bot:
[mediawiki/extensions/UniversalLanguageSelector@master] Fix language change tooltip positioning

Nikerabbit lowered the priority of this task from High to Medium.Apr 4 2017, 1:31 PM
Nikerabbit removed a project: Patch-For-Review.

Change 346362 had a related patch set uploaded (by Bartosz Dziewoński):
[mediawiki/extensions/UniversalLanguageSelector@master] Better language change tooltip positioning in 'interlanguage' mode

@Nikerabbit (on Gerrit)

This moves the tip from left to the center. This implies that in the sidebar position it will hide parts of the language list on the sidebar (and less of page content).

Sorry, I forgot that this is a thing. In sidebar position, the popup should probably be shown entirely on the right/left, rather than below. (We actually support that now!) Done in

The tip is not properly aligned with the cog in sidebar position, it is some pixels too much left at least on Chrome.

The tip ("anchor") aligns itself to the left edge of the cog ("floatable container"). It's a remnant from when the positioning logic was much dumber, and only kind of works for PopupButtonWidgets. We're changing it in in OOjs UI to align to the center.

Screenshots with both of the patches applied:

$wgULSPosition = 'personal';$wgULSPosition = 'interlanguage';
pasted_file (951×1 px, 90 KB)
pasted_file (951×1 px, 92 KB)

Change 346362 merged by jenkins-bot:
[mediawiki/extensions/UniversalLanguageSelector@master] Better language change tooltip positioning in 'interlanguage' mode