Closed Bug 376640 Opened 17 years ago Closed 17 years ago

No nsIAccessibleEvent::EVENT_FOCUS for menu item in menubar when Alt or F10 is pressed

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: aaronlev, Assigned: aaronlev)

References

(Blocks 2 open bugs)

Details

(Keywords: access, regression)

Attachments

(1 file)

When the menu bar is focused with the Alt or F10 key, no focus event is fired for the menu item. This is a regression from bug 341462, which simply checks to see if the submenu for a menu item is opened. IF not, it does not fire the focus event. We need a better fix for that bug, which checks to see if the menu item is just hovered or is capturing keypress events.
Surkov, you might want to look too, since you're working in this area of the code.
Assignee: ginn.chen → aaronleventhal
Status: NEW → ASSIGNED
Attachment #260756 - Flags: review?(ginn.chen)
Comment on attachment 260756 [details] [diff] [review]
Use menubar active state to determine if top level menuitem focus events should be fired

r=me with following changes

@@ -62,10 +62,11 @@
 #include "nsIDOMXULPopupElement.h"
 #include "nsIDocument.h"
 #include "nsIEventListenerManager.h"
-#include "nsIHTMLDocument.h"
 #include "nsIFocusController.h"
 #include "nsIFrame.h"
+#include "nsIHTMLDocument.h"
 #include "nsIInterfaceRequestorUtils.h"
+#include "nsIMenuParent.h"
Attachment #260756 - Attachment is patch: true
Attachment #260756 - Attachment mime type: application/octet-stream → text/plain
Attachment #260756 - Flags: review?(ginn.chen) → review+
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: