Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#9532 closed bug (fixed)

Need a way in Menu to keep ui-state-active class on selected item

Reported by: Felix Nagel Owned by: k_borchers
Priority: minor Milestone: git
Component: Version: git (not yet released)
Keywords: Cc:
Blocked by: Blocking:


When Selectmenu is opened we have a ui-state-focus class on the active item. This is correct as the menu item has focus but we need to add ui-state-active as well.

Without, the item is could not have its own style and looks almost similar to the button in the default CSS framework style.

I've tried to simply add the class when the menu is opened (as we did for for removing old *-focus classes) but its removed after delay in _close method. Any idea how to workaround?

This ticket is related to the comments of Juan Lanus, see:

I've already talked to Scott via IRC and we decided to open a ticket for Menu.

Change History (5)

comment:1 Changed 4 years ago by k_borchers

Owner: set to k_borchers
Status: newassigned

@fnagel, do you have, or could you create, a branch where you are adding ui-state-active but it's being removed. I have some things I can try here but would be easier to test if you have selectmenu doing what you want and demonstrating the issue with menu.

comment:2 Changed 4 years ago by Felix Nagel

@k_borchers, please see this branch: Just open a selectmenu and watch the ui-state-active class disappear after 1/3 second. The branch includes a unit test.

comment:3 Changed 4 years ago by Kris Borchers

Resolution: fixed
Status: assignedclosed

Menu: More precise removal of ui-state-active class for easier CSS overrides. Fixes #9532 - Need a way in Menu to keep ui-state-active class on selected item

Changeset: 52307ebadbca3f403549140e1d34f08bc07bf8ce

comment:4 Changed 4 years ago by Scott González

Milestone: none1.11.0

comment:5 Changed 3 years ago by Scott González

Milestone: 1.11.0git
Version: 1.10.3git
Note: See TracTickets for help on using tickets.