Page MenuHomePhabricator

Minerva on desktop doesn't show navboxes (does it?)
Closed, DuplicatePublic

Description

I could be really dumb in assuming Minerva is deployed to production as skin-for-use on desktop (even in alpha/beta), but if I go to World of Warcraft on en.WP with useformat=desktop & useskin=minerva, no navbox displays.

I would guess the LESS in hacks.less shouldn't apply to desktop or should be moved to MobileFrontend from the skin repository?

Event Timeline

Izno created this task.Jul 30 2017, 8:42 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 30 2017, 8:42 PM
Izno updated the task description. (Show Details)Jul 30 2017, 8:43 PM
Jdlrobson added a subscriber: Jdlrobson.

Yep this correct as the config variable $wgMinervaApplyKnownTemplateHacks which is set to true.
The skin should probably not be showing up in desktop preferences and is an unplanned side effect of us pulling the skin out of MobileFrontend. The skins got quite a few problems (tracked in #MinervaNeue_(Desktop) mileston , this being one of them.

I'll merge this into the parent task as we want to show this in such a way that it's useful to users regardless of their screen size : T124168

Izno added a comment.Jul 31 2017, 6:36 PM

The skin should probably not be showing up in desktop preferences and is an unplanned side effect of us pulling the skin out of MobileFrontend. The skins got quite a few problems (tracked in #MinervaNeue_(Desktop) mileston , this being one of them.

I accessed it using the link at Skin:Minerva Neue at MediaWiki wiki and just swapped the address to one I knew had a navbox at en.WP. If it is showing up in preferences that is indeed a (good) side effect of making it a true skin. :D

I'll merge this into the parent task as we want to show this in such a way that it's useful to users regardless of their screen size : T124168

That seems reasonable, but I'm skeptical there is any solution in the near term to the really small width problem with the current navbox design, which is why I filed the task specifically for desktop. I would guess the desktop-specific task would be a blocker for desktop use given that's a regression from current desktop skins--my expectation was that a task pointed specifically at the desktop would be easier to solve with a short-term fix while the bigger design question is answered.

That's fair. That particular hack could probably be displayed in a media query.
I had a look at doing this but it's non-trivial as we're using !important in our hacks.
https://en.wikipedia.org/w/index.php?title=MediaWiki:Minerva.css&action=history

In navboxes we could probably relax that... hiding it without important and then revealing it under a media query.
Happy to accept a patch to do that if you're interested in submitting one, otherwise I think T124168 is the place to put the effort.

Ahecht added a subscriber: Ahecht.Jul 31 2017, 8:53 PM

One nice side effect of removing !important is that it would allow users who want to see navboxes, despite the performance hit and glitches, to do so by adding a line to their custom.css.

Change 369453 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Relax navbox hack

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

Change 369453 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Relax navbox hack

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