Page MenuHomePhabricator

[Regression] Vector: Variant dropdown no longer triggered on click or focus
Closed, ResolvedPublic

Description

It seems the solution of bug 24298 also need to be applied to this downdrop list.


Version: 1.21.x
Severity: minor
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=24298

Details

Reference
bz46486

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:13 AM
bzimport set Reference to bz46486.

s/downdrop/dropdown/ in comment 0. :)

Tagging 'easy'. Just re-use the implementation for p-cactions.

r86861 (the fix for bug 24298) was followed-up by r99083 which made the feature more generic by applying it to any vectorMenu (instead of just p-cactions).

If this is not working on the language variants menu, perhaps it doesn't have the "vectorMenu" menu class. This would presumably cause other problems as well.

Checking SkinVector.php I see both p-varians and p-cations have class "vectorMenu" set. So it should be working.

It isn't working because I9a2ebd503e change the current-variant heading and the dropdown arrow heading (h4 and h5 respectively) to h3. And the script looks for the first h3 in a vectorMenu, which as of I9a2ebd503e is no longer the heading with the trigger in it.

Change 79847 had a related patch set uploaded by Hoo man:
Make variant selection menu toggleable by keyboard

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

Change 79847 merged by TheDJ:
Make variant selection menu toggleable by keyboard

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