Page MenuHomePhabricator

Don't show "Enable responsive mode" setting on Vector 2010
Closed, ResolvedPublic3 Estimated Story Points

Description

Description

There is a setting within preferences called Enable responsive mode. As the description within the setting says this setting "May have no effect on some skins". In order to keep preferences as clutter-free as possible, and also to avoid confusion (in the case where someone doesn't read the description), I wonder if we should:

  1. only show this setting on relevant skins
  2. remove the description text that says "May have no effect on some skins".

Event Timeline

Vecotr is the only skin that doesn't support this currently and Vector /could/ support it if we wanted.

The skins that support this are currently MonoBook, Minerva and Timeless.
The skins that don't are CologneBlue and Modern and are hidden.

I think the real problem here is the wording is not right.

Perhaps the preference could be made less confusing. All skins have a desktop version, so perhaps that could be repurposed as "Always give me desktop-mode when using my skin" and made an opt out.

That still wouldn't have any effect on Vector but it would still be true so wouldn't need that confusing text.

Maybe "Adapt layout to screen size"

Maybe "Adapt layout to screen size"

I like this description. It would be the most accurate for MediaWiki-skins-Mirage.

ovasileva triaged this task as Medium priority.Sep 28 2021, 4:56 PM

Change 726001 had a related patch set uploaded (by Ammarpad; author: Ammarpad):

[mediawiki/core@master] Reword responsive skin option help message

https://gerrit.wikimedia.org/r/726001

Test wiki created on Patch Demo by Jdlrobson using patch(es) linked to this task:

https://patchdemo.wmflabs.org/wikis/a13bbc4a89/w/

Backing up and trying to clarify what this setting does: if you turn the setting on, then access Wikipedia on your mobile phone, then request the Desktop version of the site, the setting will have turned viewport optimizing so you will see a different version of the site than you otherwise would have. If that understanding is correct I'm not sure "adapt layout to screen size makes sense" — that makes me think "give me the responsive version of this skin" (which should/does(?) happen by default...there's no non-responsive version of a skin?), but it's actually more specific/nuanced than that. It seems more like "When requesting the desktop site from a mobile device, don't optimize the viewport".

To further clarify, this setting will have no affect on your desktop experience, it is only relevant for mobile devices? If that's the case then the setting should probably mention mobile devices.

Higher level, unless we do a better job of establishing that some settings only apply to some skins I don't think we can rely on the setting description to clarify that. I think we're just setting people up for confusion.

Change 726001 merged by jenkins-bot:

[mediawiki/core@master] Reword responsive skin option help message

https://gerrit.wikimedia.org/r/726001

Jdlrobson added a subscriber: ovasileva.

@ovasileva I chatted with Alex about this and there was some confusion. I think we decided this was not a high priority and a possible distraction from desktop improvements. If that's changed maybe the three of us should sync to come up with a plan here ?

The only skins that this preference doesn't apply to are Vector, Modern and CologneBlue (and the latter two are hidden by default). It would be trivial to make this preference apply to modern Vector (not legacy) but it would also make the Vector skin responsive on mobile (which the user can opt-out of). On https://test.wikipedia.org/ all the skins abide to this preference. Personally I think this is how this should work.

To hide the preference for Vector would incur technical debt, and I don't think that would be acceptable without a plan to address this problem. There is no way to easily determine which skins this setting applies to.

We might want to think a bit more about preferences that apply to some skins as I can imagine this list growing. For example, a dark mode preference presumably would only apply to skins where a dark mode is available. I would imagine this being presented as "Use dark mode if available [] "

Test wiki on Patch demo by Jdlrobson using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/a13bbc4a89/w/

Change 881816 had a related patch set uploaded (by Func; author: Func):

[mediawiki/core@master] preferences: Only show the "responsive mode" setting for relevant skins

https://gerrit.wikimedia.org/r/881816

There is no way to easily determine which skins this setting applies to.

It's actually easy to do (at least now), you can check my patch above.

LGoto set the point value for this task to 3.Jan 26 2023, 6:39 PM

Change 881816 merged by jenkins-bot:

[mediawiki/core@master] preferences: Only show the "responsive mode" setting for relevant skins

https://gerrit.wikimedia.org/r/881816

Working as advertised on https://test.wikipedia.org/wiki/Special:Preferences#mw-prefsection-rendering
Note: it currently still shows for Modern Vector given isResponsive is declared dynamically which will be fixed as part of T106463.

Apologies for bumping, but I do notice a problem.

Is it just me, or is Vector 2022 still not displaying in responsive mode despite checking the relevant box?

Feel free to delegate to a new task if this is in fact the problem.

@Awesome_Aasim: Hard to tell without any URL provided. Please always provide clear steps to reproduce, plus try safemode. Thanks.

Is it just me, or is Vector 2022 still not displaying in responsive mode despite checking the relevant box?

Hey @Awesome_Aasim thanks for the bump!

Vector 2022 is currently not officially a responsive skin. It's not supported below 500px. The issues for this are mainly political.

We're tracking making it responsive inside T106463. The most helpful thing to move this along would be if the community drove this by approving an RFC on enwiki. Enabling this feature would mean that people who don't want a responsive skin (similar to Timeless) would need to go to Special:Preferences and uncheck the box which might be a one time disruption to their workflow.

If V22 responsive mode isn't completely ready yet, shouldn't it be declared as *not* responsive for now?

@aliu I'm not sure I understand the question:

The "responsive" setting should not show up for Vector 2022 or Vector

Screenshot 2023-11-29 at 4.14.16 PM.png (629×903 px, 132 KB)

but should show up for Timeless, Monobook and Minerva:

Screenshot 2023-11-29 at 4.14.37 PM.png (686×871 px, 141 KB)

Screenshot 2023-11-29 at 4.15.44 PM.png (386×949 px, 78 KB)

Screenshot 2023-11-29 at 4.15.14 PM.png (428×827 px, 79 KB)

For me, on selecting V10 from anything, there's an empty Skin preferences section. When I switch to another radio button after selecting v10, there is no Skin preferences section at all (as you demonstrate in your first screenshot), even though V22 has a limited width mode option. When I switch to selecting V22 from any option that isn't V10, both checkboxes are visible.

Hmmm yeh this sounds like a new bug. I wonder what changed. Could you open a new ticket for us? (If not I can do this later in the week!)

Hmmm yeh this sounds like a new bug. I wonder what changed. Could you open a new ticket for us? (If not I can do this later in the week!)

Not quite a new bug though, but it seems more reproducible than half a year ago.

Now I have a patch for it, so I created T352358 to attach to.

I also opened T352485 but I'm not sure how we fix this one...

@aliu given this issue is specific to Vector 2022 and the issue described by @Func I'd prefer to track it in the new bugs ( T352485, T352358). This initial bug was about other skins so slightly different. I can link back to here for context.

aliu renamed this task from only show "Enable responsive mode" setting on relevant skins to Don't show "Enable responsive mode" setting on Vector 2010.Nov 30 2023, 11:29 PM