Page MenuHomePhabricator and mobile.notifications.overlay modules should be ported to Minerva
Open, Needs TriagePublic


Currently MobileFrontend defines 2 Minerva skin styles for:

  • mobile.notifications.overlay

This creates a bit of a mess as it creates a cyclic dependency - MobileFrontend depends on Minerva, when it should be the other way round.

let's move these to Minerva...

Event Timeline

I believe the root problem here, is that both these modules are registered via hooks meaning MinervaNeue cannot easily patch them with skinStyles. The fact they are conditional might mean it's acceptable for the modules to search for the Minerva LESS imports.. but it feels nasty.

If they must say in the repo we'll need to work out how to do so given they both require Minerva variables:
@contentPaddingTablet, @headerHeight, @contentMargin;

The relevant code is mobile-, not Minerva-specific, though?

The skinStyles should not live in MobileFrontend. That's the main thing I would like fixed.

The longer term goal would be to move the JS modules into VisualEditor and mobile.notifications.overlay into Echo given they rely heavily on VisualEditor/Echo code any how. The behaviour of Echo and VisualEditor on Minerva should be defined and owned by those extensions.

Jdlrobson moved this task from Desktop to Tracking on the MinervaNeue board.Nov 6 2017, 9:44 PM
Jdlrobson edited projects, added MinervaNeue; removed MinervaNeue (Desktop).