Page MenuHomePhabricator

All pinned menus should automatically collapse below 1000px
Closed, ResolvedPublic3 Estimated Story Points

Description

Steps to reproduce

  1. Go to https://en.wikipedia.beta.wmflabs.org/wiki/Dog?vectorpagetools=1
  2. Make sure the main menu and page tools menu are pinned
  3. Narrow the screen to less than 1000px wide

Expected: all menus should automatically collapse
Observed: tools menu and main menu stay in a single column

AC

  • main menu, toc, and page tools auto collapse at 1000px
  • "move to sidebar" and "hide" toggle buttons in pinnable header are hidden under 1000px

QA Results - Prod

ACStatusDetails
1T326364#8576364
2T326364#8576364

Event Timeline

ovasileva created this task.
ovasileva updated the task description. (Show Details)

Change 862317 had a related patch set uploaded (by Bernard Wang; author: Jdrewniak):

[mediawiki/skins/Vector@master] Add ability to unpin pinnable headers at low resolution.

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

alexhollender_WMF renamed this task from Tools menu should automatically pin at low resolutions to All pinned menus should automatically collapse below 1000px.Jan 6 2023, 5:41 PM
alexhollender_WMF updated the task description. (Show Details)
ovasileva set the point value for this task to 3.Jan 10 2023, 5:49 PM

Change 878959 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@master] Override PinnableElement behaviour at low resolutions.

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

Jdrewniak moved this task from Doing to Code Review on the Web-Team FY2022-23 Q3 Sprint 1 board.

Change 862317 abandoned by Jdrewniak:

[mediawiki/skins/Vector@master] Add ability to unpin pinnable headers at low resolution.

Reason:

Superseded by https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/878959

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

Change 878959 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Override PinnableElement behaviour at low resolutions.

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

Change 879873 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@master] Followup to 4d1c0b89 Override PinnableElement behaviour at low resolutions.

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

Change 879873 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Followup to 4d1c0b89 Override PinnableElement behaviour at low resolutions.

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

  • main menu, toc, and page tools auto collapse at 1000px
  • "move to sidebar" and "hide" toggle buttons in pinnable header are hidden under 1000px

Looks good on beta with the exception of the empty state @alexhollender_WMF flagged. Moving to QA in prod

Test Result - Prod

Status: ✅ PASS
Environment: enwiki
OS: macOS Ventura
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

Go to https://en.wikipedia.org/wiki/Hooded_skunk
Make sure the main menu and page tools menu are pinned
Narrow the screen to less than 1000px wide
✅ AC1: main menu, toc, and page tools auto collapse at 1000px
✅ AC2: "move to sidebar" and "hide" toggle buttons in pinnable header are hidden under 1000px

Screen Recording 2023-01-31 at 1.59.46 PM.mov.gif (698×1 px, 2 MB)