Opened 13 years ago

Closed 13 years ago

Last modified 12 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: bill@…
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 doughays@… 13 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 13 years ago by Adam Peller

Cc: doughays@… added

comment:2 Changed 13 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 13 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 13 years ago by doughays@…

Attachment: 1382.patch added

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

comment:4 Changed 13 years ago by doughays@…

Please verify this is fixed after applying the supplied patch.

comment:5 Changed 13 years ago by guest

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

comment:6 Changed 13 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 13 years ago by guest

Resolution: fixed
Status: closedreopened

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

comment:8 Changed 13 years ago by doughays@…

Cc: bill@… added; doughays@… 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 13 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 12 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.