Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#1382 closed defect (fixed)

[patch][cla]MenuBar2 hijacks arrow keypresses

Reported by: guest Owned by: bill
Priority: high Milestone:
Component: Widgets Version: 0.3
Keywords: Cc: [email protected]
Blocked By: Blocking:

Description

With a MenuBar2 on a page, you can not navigate around inside of a textbox using the arrow keys. Reproducible in "tests/widget/test_Menu2_Bar.html" if you add an <input type="text"></input> to the page.

Rather than moving to the previous/next character in the textbox, the MenuBar? highlights the next/previous menu item.

Attachments (1)

1382.patch (8.1 KB) - added by [email protected] 14 years ago.
patch to Menu2.js and Menu2.css to share keystrokes w/ other document elements

Download all attachments as: .zip

Change History (11)

comment:1 Changed 14 years ago by Adam Peller

Cc: [email protected] added

comment:2 Changed 14 years ago by dylan

Milestone: 0.4
Version: 0.40.3

Did you really mean <input type="text"></input> and not <input type="text" /> ?

comment:3 Changed 14 years ago by guest

You are correct, sorry about that. Basically you just need something else on the page that you can navigate around in using the arrow keys so that you can see that the menu takes over those keys and you can no longer use them, and a textbox is a quick and simple addition.

Changed 14 years ago by [email protected]

Attachment: 1382.patch added

patch to Menu2.js and Menu2.css to share keystrokes w/ other document elements

comment:4 Changed 14 years ago by [email protected]

Please verify this is fixed after applying the supplied patch.

comment:5 Changed 14 years ago by guest

Summary: MenuBar2 hijacks arrow keypresses[patch][cla]MenuBar2 hijacks arrow keypresses

comment:6 Changed 14 years ago by bill

Resolution: fixed
Status: newclosed

(In [5562]) Fix keyboard support for menus. Fixes #1381 and also fixes #1382. Patch from Doug (IBM, CCLA on file).

comment:7 Changed 14 years ago by guest

Resolution: fixed
Status: closedreopened

This is not fixed. I am still getting "hijacked" with revision 5606.

comment:8 Changed 14 years ago by [email protected]

Cc: [email protected] added; [email protected] removed

1382.patch provided in this bug report was not correctly applied to the dojo trunk. Please reapply and then the problem will be fixed.

comment:9 Changed 14 years ago by bill

Resolution: fixed
Status: reopenedclosed

Sorry about that; I don't know what happened, but I reapplied and it's fixed for real this time in [5630].

comment:10 Changed 14 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.