| matmarex | |
| Nov 4 2017, 11:21 PM |
| F12497867: Screen Shot 2018-01-11 at 4.47.36 PM.png | |
| Jan 12 2018, 12:48 AM |
| F11786893: image.png | |
| Dec 12 2017, 12:16 PM |
| F10615335: image.png | |
| Nov 4 2017, 11:21 PM |
| F10615343: image.png | |
| Nov 4 2017, 11:21 PM |
The new ULS-based language selection dropdown is shown partially offscreen when page is in RTL language.
To reproduce:
For comparison, this is how it looks like in LTR.
Looks like the logic for displaying the popup does not flip it for RTL (it is aligned to the left edge of the dropdown, rather than the "start" edge).
Change 393098 had a related patch set uploaded (by Cparle; owner: Cparle):
[mediawiki/extensions/UploadWizard@master] Position ULS correctly for rtl languages
Change 393201 had a related patch set uploaded (by Cparle; owner: Cparle):
[mediawiki/extensions/UniversalLanguageSelector@master] Add a proxy method for css() on the jquery $menu
Change 393201 abandoned by Cparle:
Add a proxy method for css() on the jquery $menu
Reason:
See https://github.com/wikimedia/jquery.uls/pull/277 instead
@Cparle You can use the onVisible callback for adjusting any property of the ULS menu. See https://github.com/wikimedia/mediawiki-extensions-UniversalLanguageSelector/blob/master/resources/js/ext.uls.compactlinks.js#L143 for an example. You may want to do this in your uw.UlsWidget.js
You can see this flipping of ULS by clicking the button in interlanguage link of ar.wiki and en.wiki
@santhosh - that's exactly what I am doing in this patch https://gerrit.wikimedia.org/r/#/c/393098/ :)
The github patch I added is a method to allow me to query the ULS object about its css properties so that I don't have to access undocumented member variables
@santhosh - that's exactly what I am doing in this patch https://gerrit.wikimedia.org/r/#/c/393098/ :)
Thanks, I did not see this patch before commenting. I can understand your reasoning about accessing member variables.
In fact, ULS has MW specific customizations in various extensions, and MW ULS extensions. Some of those customizations are on CSS like you needed. Some of them are on the interface messages. Some of them are about changing the window behavior. So adding an accessor method of jQuery menu object alone does not solve the problem of accessing the member variables. For other use cases, you will need the localization handler, for some others you need to access the width, height calculation utilities. This is the reason for a single onVisible method that provide access to ULS objects for extensive customization.
So I am not convinced that we need to provide access to css of $menu alone with a new proxy method. @Nikerabbit thoughts?
@santhosh in the code you've linked the ULS's $menu member is accessed from the definition of CompactInterlanguageList.prototype.createSelector. I added the proxy method because I wanted to avoid accessing that member from my UploadWizard code. I'm happy to access $menu directly instead if you and @Nikerabbit think it's ok to do so
Change 393098 merged by jenkins-bot:
[mediawiki/extensions/UploadWizard@master] Position ULS correctly for rtl languages
