Page MenuHomePhabricator

Regression: "Expand all sections" setting in Minerva is broken
Closed, ResolvedPublic

Description

Broken in Id16a64573e020b4606fc9b11456163a4cd290e0b (via git blame) in wmf/1.33.0-wmf.16

Steps to reproduce

  1. on mobile enable the "Expand all sections" setting
  2. go to https://en.m.wikipedia.org/wiki/Maurice_Ravel
  3. scroll down past the first section

Expected results

  • All sections are expanded
  • Section dropdown arrows are present
  • Edit buttons are NOT present

Screen Shot 2019-03-07 at 10.00.17 AM.png (717×418 px, 65 KB)

Actual results

  • Sections are not expanded
  • Section dropdown arrows do not appear
  • Edit buttons appear

Screenshot_20190307_013003_com.android.chrome.jpg (2×1 px, 136 KB)

Environments observed

  • Browser version: Chrome 72.0.3626 / OS version: Android 9 / Device model: Honor 10
  • Browser version: Safari / OS version: iOS / Device model: iPhone 6

NOT observed on the beta cluster where it seems the sections are currently uncollapsed by default with the toggle off (same device/browser)

QA steps

Smoke test

Test on a mobile device (iOS Safari+Chrome Android suggested)

Check "memory" (i.e. that collapsed/expended sections persist after a page refresh).

Make sure using mobile device and expanded sections setting is OFF in settings.

QA Results

StatusDetails
✅ PassedT217820#5021636

Event Timeline

ovasileva moved this task from Incoming to Needs Prioritization on the Readers-Web-Backlog board.
ovasileva updated the task description. (Show Details)

Change 494889 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[mediawiki/skins/MinervaNeue@master] Passing page parameter to TOC toggler

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

Change 494889 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Passing page parameter to TOC toggler

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

Jdlrobson renamed this task from [Bug] Section dropdowns not appearing on Chrome in Minerva to [Bug] Section dropdowns not appearing on Chrome in Minerva when "expand all sections" setting is enabled.Mar 7 2019, 6:12 PM
alexhollender renamed this task from [Bug] Section dropdowns not appearing on Chrome in Minerva when "expand all sections" setting is enabled to [Bug] "Expand all sections" setting in Minerva is broken.Mar 7 2019, 6:25 PM
alexhollender updated the task description. (Show Details)
alexhollender updated the task description. (Show Details)

Change 495021 had a related patch set uploaded (by Jdlrobson; owner: Jdrewniak):
[mediawiki/skins/MinervaNeue@wmf/1.33.0-wmf.20] Passing page parameter to TOC toggler

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

Jdlrobson added subscribers: Edtadros, Jdlrobson.

Added to SWAT today 4pm PST as this is causing JavaScript client errors cc @Edtadros we'll want to QA this before then.
https://grafana.wikimedia.org/d/000000566/overview?panelId=15&fullscreen&orgId=1

Jdlrobson renamed this task from [Bug] "Expand all sections" setting in Minerva is broken to Regression: "Expand all sections" setting in Minerva is broken.Mar 7 2019, 7:47 PM
Jdlrobson added a project: Regression.
Jdlrobson updated the task description. (Show Details)

Change 495021 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@wmf/1.33.0-wmf.20] Passing page parameter to TOC toggler

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

Mentioned in SAL (#wikimedia-operations) [2019-03-08T00:23:43Z] <thcipriani@deploy1001> Synchronized php-1.33.0-wmf.20/skins/MinervaNeue/resources/skins.minerva.scripts/toc.js: SWAT: [[gerrit:495021|Passing page parameter to TOC toggler]] T217820 (duration: 00m 50s)

@Jdlrobson I wasn't sure if you wanted me to test this yet. I went through and got myself familiar with what I could. I couldn't find the "Expand all sections" setting and the beta cluster page throws an error, which I assume will be fixed when it's ready for me to test.

Sorry about the miscommunication here. The hope was to test this before the SWAT deploy on Thursday. The fix is also live on production. I've updated the QA instructions, please go ahead and test!

Test Result

Status: ✅ PASS
OS: macOS Mojave
Browser: Chrome DevTools Device Emulator (Galaxy S5)

Test Artifact(s):

QA steps

Click expanded sections
Visit https://en.m.wikipedia.org/wiki/May_14
Make sure that the arrows for collapsing are visible

T217820-1.png (1×1 px, 297 KB)

Visit the same page on beta cluster https://en.m.wikipedia.beta.wmflabs.org/wiki/May_14
Check the arrows for collapsing are visible there too

T217820-2.png (1×1 px, 267 KB)

Check "memory" (i.e. that collapsed/expended sections persist after a page refresh).
Make sure using mobile device and expanded sections setting is OFF in settings.

Visit https://en.m.wikipedia.org/wiki/May_14
Open 2 sections making a note of their headings
Births and Deaths expanded (Note: Deaths doesn't show up in the image)

T217820-5.png (16×1 px, 2 MB)

Refresh the page
Verify that the sections you opened in the previous view are still open.
Births and Deaths expanded (Note: Deaths doesn't show up in the image)

T217820-6.png (16×1 px, 2 MB)

Talked about this today. There was QA confusion caused by T159475.