286687 - Menu accelerators not exposed in MSAA

Status: RESOLVED FIXED

(Firefox :: Disability Access, defect)

Description

[Aaron Leventhal]

Windows exposes menu item names with the following format:
Label\tAccelerator

We need to add the tab and accelerator text so the screen reader can parse out
and report the keyboard accelerators.

Comment 1

[Aaron Leventhal]

Attachment: Override nsXULMenuitemAccessible::GetName() only for MSAA

Comment 2

[timeless]

Comment on attachment 177833 [details] [diff] [review]
Override nsXULMenuitemAccessible::GetName() only for MSAA

+  rv = content->GetAttr(kNameSpaceID_None, 
+				  nsAccessibilityAtoms::acceltext, accel);
			 ^ identation seems a bit off

Comment 3

[Aaron Leventhal]

Checking in accessible/src/atk/Makefile.in;
/cvsroot/mozilla/accessible/src/atk/Makefile.in,v  <--  Makefile.in
new revision: 1.12; previous revision: 1.11
done
RCS file: /cvsroot/mozilla/accessible/src/atk/nsXULMenuAccessibleWrap.h,v
done
Checking in accessible/src/atk/nsXULMenuAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/atk/nsXULMenuAccessibleWrap.h,v  <-- 
nsXULMenuAccessibleWrap.h
initial revision: 1.1
done
Checking in accessible/src/base/nsAccessibilityAtomList.h;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityAtomList.h,v  <-- 
nsAccessibilityAtomList.h
new revision: 1.15; previous revision: 1.14
done
Checking in accessible/src/base/nsAccessibilityService.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityService.cpp,v  <-- 
nsAccessibilityService.cpp
new revision: 1.131; previous revision: 1.130
done
Checking in accessible/src/mac/Makefile.in;
/cvsroot/mozilla/accessible/src/mac/Makefile.in,v  <--  Makefile.in
new revision: 1.10; previous revision: 1.9
done
RCS file: /cvsroot/mozilla/accessible/src/mac/nsXULMenuAccessibleWrap.h,v
done
Checking in accessible/src/mac/nsXULMenuAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/mac/nsXULMenuAccessibleWrap.h,v  <-- 
nsXULMenuAccessibleWrap.h
initial revision: 1.1
done
Checking in accessible/src/msaa/Makefile.in;
/cvsroot/mozilla/accessible/src/msaa/Makefile.in,v  <--  Makefile.in
new revision: 1.9; previous revision: 1.8
done
RCS file: /cvsroot/mozilla/accessible/src/msaa/nsXULMenuAccessibleWrap.cpp,v
done
Checking in accessible/src/msaa/nsXULMenuAccessibleWrap.cpp;
/cvsroot/mozilla/accessible/src/msaa/nsXULMenuAccessibleWrap.cpp,v  <-- 
nsXULMenuAccessibleWrap.cpp
initial revision: 1.1
done
RCS file: /cvsroot/mozilla/accessible/src/msaa/nsXULMenuAccessibleWrap.h,v
done
Checking in accessible/src/msaa/nsXULMenuAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/msaa/nsXULMenuAccessibleWrap.h,v  <-- 
nsXULMenuAccessibleWrap.h
initial revision: 1.1
done
Checking in accessible/src/other/Makefile.in;
/cvsroot/mozilla/accessible/src/other/Makefile.in,v  <--  Makefile.in
new revision: 1.7; previous revision: 1.6
done
RCS file: /cvsroot/mozilla/accessible/src/other/nsXULMenuAccessibleWrap.h,v
done
Checking in accessible/src/other/nsXULMenuAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/other/nsXULMenuAccessibleWrap.h,v  <-- 
nsXULMenuAccessibleWrap.h
initial revision: 1.1
done

Comment 4

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1527381