Opened 12 years ago

Last modified 12 years ago

#4933 closed defect

menu and toolbar: separat0rs can be focused with mouse click — at Version 1

Reported by: Becky Gibson Owned by:
Priority: high Milestone: 1.1
Component: Dijit Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Becky Gibson)

You can focus a MenuSeparator? and a ToolbarSeparator? and should not be allowed to. This is because these widgets get assigned a tabindex=-1 when added to the menu and toolbar. Did a quick fix for 1.0 to prevent an error (see #4927) but need to create a more complete solution.

The problem can be seen if you focus a toolbar button, then click on a toolbar separator that is not close to the button. The separator gets focus and the previously focused button loses focus indication. If you then press a right arrow key, focus will go to the button that was next to the previously focused button. Which is what we want but is visually odd.

The same problem occurs in the context menu but is not as evident since the menu item which was focused before the separator is clicked still "looks" focused.

The solution is to not allow focus to the separators.

Change History (1)

comment:1 Changed 12 years ago by Becky Gibson

Description: modified (diff)
Note: See TracTickets for help on using tickets.