Page MenuHomePhabricator

"More" menu in Vector is no longer accessible except by mouse hover
Closed, ResolvedPublic

Description

"More" menu in Vector is no longer accessible except by mouse hover. It should also be possible to open it by clicking it, or by keyboard.

Solution

The menu's <ul> was wrapped into a <div>: patch 589395 task T249372.

The checkbox hack's CSS selector needs to be updated with the new sibling.

QA Results - Beta

ACStatusDetails
1T255069#6222561
2T255069#6222561 PLEASE SEE NOTE

QA Results - Prod

ACStatusDetails
1T255069#6245066
2T255069#6245066 PLEASE SEE NOTE

Event Timeline

This has been deployed in wmf.34. Reported on May 30: T249372#6178392

Jdlrobson lowered the priority of this task from Unbreak Now! to High.Jun 10 2020, 9:19 PM

Not unbreak now status given wmf.34.

Duplicate of T254851?

Sounds reasonable, but that's not the case. On version 1.34.1 focus + SPACE toggles the checkbox, but not ENTER. This behavior did not change recently.

It should also be possible to open it by clicking it, or by keyboard.

The checkbox hack's CSS selector broke when the <ul> was wrapped into a <div>.
I have deja vu. Somewhere I've written this should be expected.

Change 605201 had a related patch set uploaded (by Aron Manning; owner: Aron Manning):
[mediawiki/skins/Vector@master] Fix dropdown menu checkbox hack's CSS selector

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

Change 605201 merged by jenkins-bot:
[mediawiki/skins/Vector@master] Fix dropdown menu checkbox hack's CSS selector

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

Thanks, that makes it work for me locally again.

Edtadros subscribed.

Test Result - Beta

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

Test Artifact(s):

QA steps

✅ AC1: "More" menu should expend when hovered over.

Conflict-title-0.5294066344342192-Iñtërnâtiônàlizætiøn - Wikipedia, the free encyclopedia.gif (862×510 px, 3 MB)

✅ AC2: More menu should be expanded by mouse click and keyboard

NOTE: While the More menu does expand when clicked or space is entered on the keyboard. When you click it and then click it to collapse again, even though the arrow switches to the uncollapsed state, the actual menu doesn't collapse.

Conflict-title-0.5294066344342192-Iñtërnâtiônàlizætiøn - Wikipedia, the free encyclopedia (1).gif (828×500 px, 3 MB)

When you click it [the More menu] and then click it to collapse again, even though the arrow switches to the uncollapsed state, the actual menu doesn't collapse.

Yes, that's an implementation shortcoming: the mouse is still hovering the menu thus it remains open until the mouse moves out. It's very weird, but it was like that for a very long time.

wmf.37 reached group 2 on 2020-06-17.
This passes ad-hoc testing on enwiki, dewiki, enwikt, meta, mediawiki.
@Edtadros unless you want to do extra testing, I went ahead and boldly closed this.

@Demian, thanks for taking the initiative, below are the documentation and test artifacts for the testing

Test Result - Prod

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

Test Artifact(s):

QA steps

✅ AC1: "More" menu should expend when hovered over.

Wikipedia, the free encyclopedia (2).gif (480×436 px, 2 MB)

✅ AC2: More menu should be expanded by mouse click and keyboard

NOTE: While the More menu does expand when clicked or space is entered on the keyboard. When you click it and then click it to collapse again, even though the arrow switches to the uncollapsed state, the actual menu doesn't collapse.

Wikipedia, the free encyclopedia (3).gif (480×436 px, 3 MB)