Closed
Bug 376640
Opened 18 years ago
Closed 18 years ago
No nsIAccessibleEvent::EVENT_FOCUS for menu item in menubar when Alt or F10 is pressed
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
References
(Blocks 2 open bugs)
Details
(Keywords: access, regression)
Attachments
(1 file)
|
(deleted),
patch
|
ginnchen+exoracle
:
review+
|
Details | Diff | Splinter Review |
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.
|
Assignee | |
Comment 1•18 years ago
|
||
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+
|
|
Assignee | |
Updated•18 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•