@Isarra would you adopt this if added ?

In T258290#6315848, @Jdlrobson wrote:

@Isarra would you adopt this if added ?

Every responsive skin I maintain uses the same viewport, and if it doesn't, it usually winds up on the same sooner or later out of paranoia, as I don't know where any of them even came from and I don't understand how it works, just that this one apparently does. So being able to just grab something from mw instead would probably make a bit more sense, yes.

the option defaults to false

Might make more sense to do it the other way around, as html is by default responsive and it's only when you start doing weird styles with it that it stops being so. Modern skin developers are hopefully not doing this, as mobile is now a primary target (it was not when monobook et all were made, and as those were also targetting things like IE, they really did have to do some weird things to get their layouts to work at all at times), and it would be best not to penalise them for following best practices. So I would recommend making it true by default, and then explicitly marking the older skins as false.

Would it be possible to do it this way while still allowing the current method of adding it to override it, for backwards compatibility?

Would it be possible to do it this way while still allowing the current method of adding it to override it, for backwards compatibility?

Making it the default would definitely be possible, but it would be a breaking change and need to be done in stages - adding the method and defaulting to false, updating all skins (at least Modern, CologneBlue and Monobook) to use or not use it, and then defaulting it to true.

It would be much easier to default it to true on the SkinMustache's constructor which is a new class and doesn't suffer from the legacy problems of other skins - which Minerva and Vector will be adopting - but I'm not sure if Timeless has plans to adopt it? (if only to get away from PHP string stitching and having multiple classes for a skin).

Here's Vector's adoption of it for example which will hopefully land in the next 2 weeks: https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/593070 - is that something Timeless would be interested in adopting in some form on the long term?

Are there any simpler examples? Do you have any plans to migrate the Example skin?

In T258290#6315890, @Isarra wrote:

Are there any simpler examples? Do you have any plans to migrate the Example skin?

I'd love to migrate the Example skin to show what that would look like. I was just a little reluctant too as I'm not sure who would review and merge :)
The Modern port is probably the simplest example right now, but it also requires dependencies in Vector to work:

https://gerrit.wikimedia.org/r/c/mediawiki/skins/Modern/+/585834

I've made a note to do better with this regard - T258292
I know @Mainframe98 has been working on a skin (possibly using SkinMustache?) but I don't know if there's anything shareable there (yet)

In T258290#6315907, @Jdlrobson wrote:

I know @Mainframe98 has been working on a skin (possibly using SkinMustache?) but I don't know if there's anything shareable there (yet)

Migrating as we speak (write), but got stuck on something I haven't had the time to investigate into yet. Once that's done I'll see what I can do to upload a prototype.
In any event, if you need help, feel free to add me to any patches.

I've also defined the viewport attribute, but with only width=device-width, initial-scale=1.0, user-scalable=yes set; I'm not exactly sure what the difference is.

@Mainframe @Isarra given Minerva is more stress tested (more users) I'd personally suggest using its value for the viewport attribute, unless you can suggest reasons why we would not want to use it.

initial-scale=1.0, user-scalable=yes, minimum-scale=0.25, maximum-scale=5.0, width=device-width

I can't recall the reasons for using minimum and maximum scale but am open to dropping those if we feel strongly about it.

Look, as long as it works, I'm fine with any sane default. :P

Change 618132 had a related patch set uploaded (by Mainframe98; owner: Mainframe98):
[mediawiki/core@master] Skin: Add a skin option for responsiveness

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

Change 618133 had a related patch set uploaded (by Mainframe98; owner: Mainframe98):
[mediawiki/skins/Timeless@master] Specify the responsive option rather than manually specifying viewport

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

Change 618134 had a related patch set uploaded (by Mainframe98; owner: Mainframe98):
[mediawiki/skins/MinervaNeue@master] Specify the responsive option rather than manually specifying viewport

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

Change 618135 had a related patch set uploaded (by Mainframe98; owner: Mainframe98):
[mediawiki/skins/MonoBook@master] Specify the responsive option rather than manually specifying viewport

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

Change 618132 merged by jenkins-bot:
[mediawiki/core@master] Skin: Add a skin option for responsiveness

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

Change 618134 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Specify the responsive option rather than manually specifying viewport

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

Change 618135 merged by jenkins-bot:
[mediawiki/skins/MonoBook@master] Specify the responsive option rather than manually specifying viewport

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

Change 618133 merged by jenkins-bot:
[mediawiki/skins/Timeless@master] Specify the responsive option rather than manually specifying viewport

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

The double dash is kind of not in keeping with the style of the other selectors (in general as well as the classes on body). Can a quick followup be done to use one dash instead? Or some rationale provided for why two were used?

Per @Jdlrobson's comment on the patchset:

we don't often use '--' and use '-' if a skin is called responsive it will get the class 'skin-responsive' so this ensures it won't be targetted

Change 622469 had a related patch set uploaded (by Samwilson; owner: Samwilson):
[mediawiki/skins/Timeless@master] Update required core version

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

Change 622469 merged by jenkins-bot:
[mediawiki/skins/Timeless@master] Update required core version

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