Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#15157 closed bug (fixed)

Can focus menu dividers with the keyboard

Reported by: klmcdorm Owned by: Scott González
Priority: minor Milestone: 1.12.2
Component: ui.menu Version: 1.12.1
Keywords: Cc:
Blocked by: Blocking:

Description

jsFiddle:https://jsfiddle.net/bmu3k2w3/2/
If a menu has a divider as the first or last item, it can be focused using the keyboard. Reproduction steps:

  1. Click on the input box and press Tab to focus on the menu.
  2. Move focus to the last menu item by pressing the down arrow key.
  3. Try to move focus back to the first menu item by pressing down again.
  4. Focus moves to the divider before the first item instead. In the menu's focus event, ui.item is the divider.


This also happens when moving up from the first item, or when using Page Up or Page Down.
Tested on Windows 7 with Chrome 56.0.2924.87, Firefox 52.0.1, and Internet Explorer 11.0.9600.18537.

Change History (4)

comment:2 Changed 6 years ago by Scott González

Status: newopen

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

Owner: set to Scott González
Resolution: fixed
Status: openclosed

In a3e953b:

Menu: Don't focus dividers when wrapping via keyboard navigation

Fixes #15157
Closes gh-1804

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

Milestone: none1.12.2
Note: See TracTickets for help on using tickets.