Opened 7 years ago
Closed 7 years ago
#14945 closed bug (fixed)
ui.menu blur event ui item returns null
Reported by: | Andrea Fercia | Owned by: | Scott González |
---|---|---|---|
Priority: | blocker | Milestone: | 1.12.0 |
Component: | ui.menu | Version: | 1.11.3 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Looking at the docs, they state the ui.menu blur event passes the event itself and a ui object containing the currently active menu item. See https://api.jqueryui.com/menu/#event-blur
By the way, the code actually returns null
for the ui.item. Seems either the docs, but more likely the null value is wrong. I can understand why the active menu item is set to null
but maybe the event should be triggered before?
FWIW noticed this while working on a new core feature for WordPress 4.5, see https://core.trac.wordpress.org/attachment/ticket/33301/33301.voiceover.diff where we used a workaround. Any feedback more than welcome, thanks! :)
Change History (2)
comment:1 Changed 7 years ago by
Milestone: | none → 1.12.0 |
---|---|
Priority: | minor → blocker |
Status: | new → open |
comment:2 Changed 7 years ago by
Owner: | set to Scott González |
---|---|
Resolution: | → fixed |
Status: | open → closed |
In 57f801f:
Yeah, this is a bug in the code. And a pretty obvious bug looking at the code :-/