In urdu wikipedia, we noticed here on gadgets definition page that
targets=mobile
targets both mobile and desktop users. While we would like to enable this gadget for mobile users only.
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T158181 Aim for workflow equivalence for MediaWiki on desktop and mobile web | |||
| Resolved | Jdlrobson | T127268 Dismantle ResourceLoader's "targets" system | |||
| Duplicate | None | T127147 Gadget definition "targets=mobile" seem to target mobile and desktop users on ur.wp |
Event Timeline
Comment Actions
While https://ur.wikipedia.org/wiki/میڈیاویکی:Gadget-NotoNastaleeqMobile.css exists, very similar CSS font definitions are also in https://ur.wikipedia.org/wiki/میڈیاویکی:Common.css .
So I could imagine that the last link defines the fonts and that there is no software bug here. (But I don't know what gets loaded first and is supposed to be "stronger" / overwrite.) Could you confirm?
Comment Actions
On mobile browsers, Gadget should get loaded first and overwrite the fonts defined in common.css.
Comment Actions
Steps to repro:
- Open https://ur.wikipedia.org/wiki/خاص:ترجیحات?uselang=qqx#mw-prefsection-gadgets
- Open Gadgets tab, make sure (gadget-NotoNastaleeq) is NOT checked but check (gadget-NotoNastaleeqMobile) then save gadgets preference.
- Open https://ur.wikipedia.org/wiki/صفحۂ_اول
Actual:
Black background on Urdu Wikipedia with the gadget.
Expected:
No black background on Urdu Wikipedia as ".skin-vector { background-color: black; }" was applied on https://ur.wikipedia.org/wiki/میڈیاویکی:Gadget-NotoNastaleeqMobile.css which is a mobile gadget and not anywhere else, see https://ur.wikipedia.org/wiki/میڈیاویکی:Gadgets-definition which notes NotoNastaleeqMobile targets mobile, "NotoNastaleeqMobile[ResourceLoader|top|targets=mobile]|NotoNastaleeqMobile.css"
Comment Actions
I've moved the test on Test Wikipedia to not confuse Urdu Wikipedia users, so new steps to repro:
- Open https://test.wikipedia.org/wiki/Special:Preferences#mw-prefsection-gadgets and enable "Test mobile only gadget bug" and save
- Open https://test.wikipedia.org/wiki/Main_Page
Actual:
Black sidebar background
Expected:
It shouldn't be. https://test.wikipedia.org/wiki/MediaWiki:Gadget-mobileonlygadget.css was "mobileonlygadget[ResourceLoader|targets=mobile]|mobileonlygadget.css" on https://test.wikipedia.org/wiki/MediaWiki:Gadgets-definition , perhaps the expected behavior would not applying it on Desktop view.
Comment Actions
Looks like maybe Gadgets doesn't implement the requirements for Targets completely. It added support for setting the module object properties but doesn't read it back out when adding modules to the page.
Either that, or the bug is in MediaWiki core OutputPage, where the style queue is missing support for the target restriction (the stylesheet urls lacks target=). This is one of various aspects of the Target system that wasn't fully considered when it was implemented. However this will be obsolete with T127268, so I don't expect these bugs to be addressed in core (unless volunteered).
Per T127268, I'd encourage you to plan ahead for when the targets system doesn't exist anymore.
For example, you may want to use the skins filter instead (e.g. skins=minerva).
Comment Actions
Too bad that doesn't work either. Gadgets uses the user's skin setting, which is never minerva.
Comment Actions
@Majr was working on it on T140863: Allow mobile-specific gadgets: https://gerrit.wikimedia.org/r/#/c/299950/
I'm not sure if this is still relevant, given this comment: T173309#3554105
Comment Actions
It's a simple fix, but it was stalled waiting review. If the issue is still relevant a year and half later and someone wants to review I'm happy to rebase. The main thing I personally wasn't sure of was if my test was any good or made any sense.
Comment Actions
Yikes, this still didn't get merged i guess..
I rebased the patch and forked the specific issue to T199478: Gadgets should use the current skin, instead of the skin preference, since it doesn't really fix the targets system, which this ticket is about.