| • alexhollender_WMF | |
| Mar 2 2021, 5:58 PM |
| F34444354: Screen Shot 2021-05-06 at 4.48.54 PM.png | |
| May 7 2021, 12:00 AM |
| F34444353: Screen Shot 2021-05-06 at 4.49.33 PM.png | |
| May 7 2021, 12:00 AM |
| F34432320: Screen Shot 2021-04-28 at 9.48.16 AM.png | |
| Apr 28 2021, 4:50 PM |
| F34392686: Screen Shot 2021-04-14 at 9.58.43 AM.png | |
| Apr 14 2021, 5:02 PM |
| F34131881: Screen Shot 2021-03-02 at 1.35.19 PM.png | |
| Mar 2 2021, 5:58 PM |
| F34131896: Screen Shot 2021-03-02 at 1.56.38 PM.png | |
| Mar 2 2021, 5:58 PM |
| F34131891: Screen Shot 2021-03-02 at 1.54.59 PM.png | |
| Mar 2 2021, 5:58 PM |
| F34131894: Screen Shot 2021-03-02 at 1.55.35 PM.png | |
| Mar 2 2021, 5:58 PM |
Currently the language menu is positioned to the left of the language button. It should be positioned below, and right-aligned with, the language button:
| current | ideal |
|---|---|
 |  |
Here is how the various sizes of the menu would look with the new positioning:
| 4 columns | 2 columns | 1 column |
 |  |  |
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T276248#7042697 |
| 2 | ✅ | T276248#7042697 |
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T276248#7068223 |
| 2 | ✅ | T276248#7068223 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | ovasileva | T275766 Log sessionID, isAnon, editBucketCount, and selectedLanguage properties in the UniversalLanguageSelector instrument | |||
| Resolved | ovasileva | T275794 Log time on page before switching language as part of the UniversalLanguageSelector instrument | |||
| Resolved | ovasileva | T256023 [EPIC] Improve language switching capabilities | |||
| Resolved | ovasileva | T268241 Style language button and menu | |||
| Resolved | ovasileva | T269093 Deploy new language switching location to test wikis and begin A/B test pt 1 | |||
| Resolved | ovasileva | T280788 Post-deployment QA for language switching functionality | |||
| Resolved | Jdrewniak | T280526 Deploy new language switching functionality to logged-in users | |||
| Resolved | ovasileva | T276248 Position language menu below language button |
@santhosh I notice on Wikimedia Commons the ULS dialog appears below the button. Is this something that can be configured in ULS?
@ovasileva let's add this to our next sync with the language team.
Change 672738 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/UniversalLanguageSelector@master] Change ULS content language selector positioning
Change 672738 merged by jenkins-bot:
[mediawiki/extensions/UniversalLanguageSelector@master] Change ULS content language selector positioning
Change 672979 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/UniversalLanguageSelector@master] Improve positioning for content language selector on narrow screens
Change 672979 merged by jenkins-bot:
[mediawiki/extensions/UniversalLanguageSelector@master] Improve positioning for content language selector on narrow screens
During demo to Language team I noticed the positioning of input/display settings may go off-screen when opened from the new button. I'll see if I can fix that easily.
And another issue is that on narrow screens the dialog changes width shortly after it is shown. This is because the input/display settings are loaded asynchronously and once they inject their own buttons, the dialog becomes wider to accommodate them. There is a trade off here on avoiding the flash, or delaying its appearance under those modules are loaded. I guess maybe some trickery we could fix the width in a way that it doesn't change, but I don't have good ideas for that.
Change 673848 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/UniversalLanguageSelector@master] Avoid content language selector dialog resize shortly after opening
Change 673959 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/UniversalLanguageSelector@master] Fix positioning of input/display settings for new language selector
Change 673959 merged by jenkins-bot:
[mediawiki/extensions/UniversalLanguageSelector@master] Fix positioning of input/display settings for new language selector
Change 673848 merged by jenkins-bot:
[mediawiki/extensions/UniversalLanguageSelector@master] Avoid content language selector dialog resize shortly after opening
@Jdlrobson In beta I can only see the 2 column version. I can confirm that the menu appears below the language button. Is there is a way to force the 1 and 4 columns? I get a two-column even with zero languages so I'm not sure if the number of languages is what would make the change.
@Edtadros you can use https://patchdemo.wmflabs.org/wikis/8e7e015a1d/wiki/Main_Page to test.
@Jdlrobson, If i go to patchdemo everything looks right, but that's expected so I'm not sure that's more than just a second pair of eyes on it. Ignoring the multi-columns shown in the description and on patchdemo, on Beta the language menu is not positioned below or right aligned to the language button.
On beta cluster, it seems the mw-interlanguage-selector class has moved from the button to h3. The dialog is correctly positioned against the h3 now, but that's not what is desired. Any idea why/when this changed?
On beta cluster, it seems the mw-interlanguage-selector class has moved from the button to h3
The new language button has never actually used a button tag and the mw-interlanguage-selector has always been on an h3. What has changed is the styling rules for mw-ui-button to make sure text is centered.
It seems like the ULS dialog is using the span for positioning?
Change 681761 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/extensions/UniversalLanguageSelector@master] Add margin to modern Vector ULS dialog
Change 681761 abandoned by Jdlrobson:
[mediawiki/extensions/UniversalLanguageSelector@master] Add margin to modern Vector ULS dialog
Reason:
yip
Change 681762 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/extensions/UniversalLanguageSelector@master] Use ev.currentTarget not ev.target
Looking close, this was a problem because span.selector-open was being used as the $trigger inside ext.uls.launch instead of the heading... this should fix the problem.
Change 681762 merged by jenkins-bot:
[mediawiki/extensions/UniversalLanguageSelector@master] Use ev.currentTarget not ev.target
@Nikerabbit please let me know if you need any follow ups and I will take care of those first thing tomorrow.
Status: ✅ PASS
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA
Test Artifact(s):
✅ AC1: Language menu should be position below the language button
✅ AC2: Language menu should be right-aligned with the language button.
Status: ✅ PASS
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA
Test Artifact(s):
✅ AC1: Language menu should be position below the language button
✅ AC2: Language menu should be right-aligned with the language button.
