Page MenuHomePhabricator

[Sticky header] Adjust position of ULS menu
Closed, ResolvedPublic2 Estimated Story Points

Description

Description

Currently the ULS is about 22px below the ULS button. It should be closer to the button. The user menu + user menu button should be used as a reference for the correct spacing.

too much spacecorrect
image.png (934×2 px, 260 KB)
Screen Shot 2022-01-08 at 1.23.14 PM.png (854×1 px, 75 KB)

QA Results - Beta|Prod

ACStatusDetails
1T298835#7695611 this is a pass per T298835#7696927

Event Timeline

alexhollender_WMF renamed this task from [Sticky header] Adjust position of ULS to [Sticky header] Adjust position of ULS menu.Jan 10 2022, 6:08 PM

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

[mediawiki/skins/Vector@master] Ensure sticky header ULS menu appears in default location right underneath the button

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

bwang removed bwang as the assignee of this task.Jan 19 2022, 7:39 PM
bwang moved this task from Doing to Code Review on the Web-Team-Backlog (Kanbanana-FY-2021-22) board.
bwang subscribed.

Change 755474 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Ensure sticky header ULS menu appears in default location right underneath the button

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

Edtadros subscribed.

Test Result - Beta

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

Test Artifact(s):

QA Steps

❓ AC1: Currently the ULS is about 22px below the ULS button. It should be closer to the button. The user menu + user menu button should be used as a reference for the correct spacing.

@alexhollender_WMF, There is a slightly bigger gap between the blue outline on the ULS and the menu as compared to the user menu button and the user menu. Since the ULS doesn't stay outlined when you click it, I used the super scientific method of clicking it rapidly to see if there was a gap. Is this slightly larger gap a pass? Also is the lack of outline when the ULS is clicked an issue?

Screen Recording 2022-02-08 at 3.40.06 PM.mov.gif (112×442 px, 83 KB)

Screen Shot 2022-02-08 at 3.34.08 PM.png (333×405 px, 39 KB)

Screen Shot 2022-02-08 at 3.33.59 PM.png (419×489 px, 30 KB)

@Edtadros thanks for the detailed review here, we can consider this a pass.

  • the lack of a blue outline when clicking the language button is expected
  • the difference in the spacing is acceptable

Verified in production. Resolving.
QA:

Screen Shot 2022-02-17 at 7.33.32 PM.png (432×840 px, 56 KB)

Menu is less than 22px away from the button