Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3318 closed defect (fixed)

PopupMenu positioned incorrectly when opened with the keyboard

Reported by: simonjb Owned by: simonjb
Priority: high Milestone: 0.9
Component: Accessibility Version: 0.9
Keywords: Cc: Becky Gibson, bill
Blocked By: Blocking:

Description

To reproduce:

  1. open dijit/tests/test_Menu.html
  2. focus the text area containing the text "hello there!"
  3. open the menu (on windows: shift-f10 or windows context menu key, on mac FF: ctrl-space)

Expected:

  • menu to open just below and right of the top left corner of the text area

Actual (FF):

  • menu opens at the top left corner of the browser window

Actual (IE):

  • menu opens at the mouse cursor or not at all if the mouse cursor is positioned off the right or bottom edge of the browser window

Attachments (1)

ContextMenuPositioning.diff (3.8 KB) - added by simonjb 12 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 12 years ago by simonjb

Cc: Becky Gibson bill added; becky bill removed

comment:2 Changed 12 years ago by simonjb

Fixing this bug is especially important for zoomtext users.

comment:3 Changed 12 years ago by bill

OK, so the implication is that if you use the keyboard to open the menu, it should be placed relative to the currently focused node, rather than relative to the mouse position? I guess that makes sense. You should be able to get the currently focused node from FocusManager?.

comment:4 Changed 12 years ago by simonjb

Yes, exactly. If the menu is opened via the keyboard, it should be placed relative to the currently focused node.

comment:5 Changed 12 years ago by simonjb

Status: newassigned

Changed 12 years ago by simonjb

Attachment: ContextMenuPositioning.diff added

comment:6 Changed 12 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [9147]) Correctly position context menu when opened via keyboard (shift-F10 on windows). Fixes #3318 Patch by Simon Bates (CLA on file)

comment:7 Changed 12 years ago by bill

(In [9148]) Standardize call signatures for dijit.util.popup.open*() methods, refs #3318.

Note: See TracTickets for help on using tickets.