Page MenuHomePhabricator

[Regression, Visual refinements] When main menu is collapsed table of contents jumps on page load
Closed, ResolvedPublic2 Estimated Story Points

Description

Go to https://en.wikipedia.beta.wmflabs.org/wiki/Albert_Einstein
Close the sidebar, and wait for the preference to save
Hard reload the page.

What happens:

collapsejiggle.gif (505×1 px, 199 KB)

Expected:
The table of contents should not change position.

QA Results - Beta

ACStatusDetails
1T316964#8239990

QA Results - Prod

ACStatusDetails
1T316964#8239997

Event Timeline

@bwang the issue seems to be the rule .vector-toc-not-collapsed #mw-sidebar-checkbox:not(:checked) ~ .mw-table-of-contents-container .vector-sticky-toc-container ...

@alexhollender_WMF - assuming this is a blocker for deployment. Thoughts from your side?

@Jdlrobson nice catch.

@alexhollender_WMF - assuming this is a blocker for deployment. Thoughts from your side?

ya, i agree

@bwang the issue seems to be the rule .vector-toc-not-collapsed #mw-sidebar-checkbox:not(:checked) ~ .mw-table-of-contents-container .vector-sticky-toc-container ...

Yep, we are adding that class via JS right now when it should be serverside

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

[mediawiki/skins/Vector@master] Add vector-toc-not-collapsed class serverside

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

Change 830942 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Add vector-toc-not-collapsed class serverside

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

Please move to QA or needs more work.

alexhollender_WMF renamed this task from [Regression] When sidebar is collapsed table of contents jumps on page load to [Regression] When main menu is collapsed table of contents jumps on page load.Sep 9 2022, 3:15 PM

I am no longer seeing the TOC jump down on page load when the main menu is collapsed

Jdlrobson renamed this task from [Regression] When main menu is collapsed table of contents jumps on page load to [Regression, Visual refinements] When main menu is collapsed table of contents jumps on page load.Sep 13 2022, 11:33 PM

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Monterey
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

Go to https://en.wikipedia.beta.wmflabs.org/wiki/Albert_Einstein
Close the sidebar, and wait for the preference to save
Hard reload the page.
✅ AC1: The table of contents should not change position.

Screen Recording 2022-09-15 at 9.29.02 AM.mov.gif (828×620 px, 410 KB)

Edtadros subscribed.

Test Result - Prod

Status: ✅ PASS
Environment: cawiki
OS: macOS Monterey
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

Go to https://ca.wikipedia.org/wiki/Alcarr%C3%A0s_(pel%C2%B7l%C3%ADcula)
Close the sidebar, and wait for the preference to save
Hard reload the page.
✅ AC1: The table of contents should not change position.

Screen Recording 2022-09-15 at 9.34.54 AM.mov.gif (828×620 px, 279 KB)