Page MenuHomePhabricator

Sticky user menu is invisible but still clickable after scrolling to top
Closed, ResolvedPublic1 Estimated Story PointsBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Open a page in Vector 2022
  • Scroll down so the sticky header appears
  • Open the user menu
  • Scroll to top so the the sticky header disappears
  • Hover the mouse over where the user menu was

What happens?:
The user menu is invisible (transparent) but lays over the page content and the links can still be clicked.

What should have happened instead?:
The user menu actually disappears.

QA Results - Beta

ACStatusDetails
1T325083#9503405

QA Results - Prod

ACStatusDetails
1T325083#9540481

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
ovasileva triaged this task as Medium priority.Mar 15 2023, 4:53 PM
ovasileva moved this task from Not ready to estimate to Current Quarter on the Web-Team-Backlog board.
ovasileva lowered the priority of this task from Medium to Low.Apr 17 2023, 5:48 PM
LGoto set the point value for this task to 3.Apr 17 2023, 5:51 PM
Jdlrobson raised the priority of this task from Low to Medium.May 9 2023, 5:00 PM

As I have said in T336159, .uls-dialog-sticky-hide #vector-sticky-header { display: none; } should solve the problem. I was testing the new Vector skin these days and got "clickjacked" for, like, three times. Hard to imagine how many editors are affected by this.

ovasileva raised the priority of this task from Medium to High.Jan 11 2024, 6:19 PM

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

[mediawiki/skins/Vector@master] Deactivate sticky header menus and search when sticky header is hidden

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

bwang removed bwang as the assignee of this task.Jan 18 2024, 8:57 PM
bwang subscribed.
bwang changed the point value for this task from 3 to 1.Jan 29 2024, 6:45 PM

Change 991646 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Deactivate sticky header menus and search when sticky header is hidden

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

Jdlrobson added subscribers: Edtadros, Jdlrobson.

@bwang I missed something in code review - which explains why Pixel was not working - it was hitting a JavaScript error and cryptically reporting it.

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

[mediawiki/skins/Vector@master] Deactivate sticky header menus and search when sticky header is hidden attempt 2

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

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

[mediawiki/skins/Vector@master] Deactivate sticky header menus and search when sticky header is hidden attempt 2

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

bwang removed bwang as the assignee of this task.Jan 30 2024, 6:14 PM
bwang moved this task from Doing to Code Review on the Web-Team-Backlog (FY2023-24 Q3 Sprint 2) board.

Change 994145 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Deactivate sticky header menus and search when sticky header is hidden attempt 2

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

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

Open a page in Vector 2022
Scroll down so the sticky header appears
Open the user menu
Scroll to top so the the sticky header disappears
Hover the mouse over where the user menu was
✅ AC1: The user menu actually disappears, it does not over the page content with links that can still be clicked.

screenshot 220.mov.gif (564×1 px, 1 MB)

Test Result - Prod

Status: ✅ PASS
Environment: enwiki
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

Open a page in Vector 2022
Scroll down so the sticky header appears
Open the user menu
Scroll to top so the the sticky header disappears
Hover the mouse over where the user menu was
✅ AC1: The user menu actually disappears, it does not over the page content with links that can still be clicked.

screenshot 250.mov.gif (1×820 px, 685 KB)