Page MenuHomePhabricator

Promote expand all sections feature from beta (in feature branch)
Closed, ResolvedPublic1 Estimated Story Points

A feature "expand all sections" allows users to always keep sections open. It works for anonymous users.

Screen Shot 2017-06-29 at 11.09.50 AM.png (565×929 px, 51 KB)

Acceptance criteria

Promote "expand all sections" to stable:

  • remove from list of beta features and display expand all sections within list of settings

Developer notes

The MFExpandAllSectionsUserOption controls this. Current value is:

"MFExpandAllSectionsUserOption": {
        "base": false,
        "beta": true

base must be flipped to true either via change inside repo or SWAT

Event Timeline

Did we make a decision? Are we moving this to stable? The description hasn't been updated...

Does this deploy to all wikis? Do we need any extra logging? Will this trigger existing expand / collapse logging for all sections on a page?

Jdlrobson set the point value for this task to 1.

Being bold and assigning 1 since this is essentially a swat deploy.

Change 396284 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@specialpages] Promote expand all sections and fontchanger to stable

ovasileva renamed this task from Promote expand all sections feature from beta to Promote expand all sections feature from beta (in feature branch).Dec 11 2017, 3:36 PM

Change 396284 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@specialpages] Promote expand all sections and fontchanger to stable

Overall settings work seems to be working as expected! yay!

but one minor issue on this card.... kind of a bug to "expand al sections" feature so let me know if we want to create a follow up card or fix it as part of this card


Tablets/desktop layout is set to expanding all sections by default on MinervaNeue. this creates a little bit of problem when a setting to enable/disable this feature is introduced on tablets. as a result, if you turn off "expand all sections" on tablets, it won't actually collapse all section which is the expected behaviour.

what should happen

  • on tablets, this particular toggle should be switched 'ON' by default
  • on tablets, turning off this toggle in settings should collapse all sections on tablets

as per @pmiazga created a subtask to this for the above issue -

T183493: Remove 'Expand all section' toggle on tablet and desktop mode

cc @ovasileva if T183493 should be a blocker to promote this feature to stable

I'll be bold, we have an UI glitch, we cannot promote to stable something that is broken. Moving to blocked, as it's blocked on T183493.

@Jdlrobson it would be amazing if you can verify the T183493 and propose a fix.

Since has been merged and we have T183493 and the deploy task this particular task can be considered done. Maybe Nirzar should sign off?