280248 - Expose <q> to MSAA

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

We currently expose blockquote, but we forgot to add q.

Needed for Window-Eyes support.

Comment 1

[Aaron Leventhal]

Attachment: Add ::q to if () for generic accessible creation. Atom was already defined.

Comment 2

[Philip K. Warren]

Comment on attachment 172734 [details] [diff] [review]
Add ::q to if () for generic accessible creation. Atom was already defined.

>diff -p -u -5 -r1.123 nsAccessibilityService.cpp
>--- accessible/src/base/nsAccessibilityService.cpp	26 Jan 2005 22:41:05 -0000	1.123
>+++ accessible/src/base/nsAccessibilityService.cpp	28 Jan 2005 20:56:39 -0000
>@@ -450,10 +450,11 @@ nsAccessibilityService::CreateHTMLAccess
> #endif
>   else if (content->HasAttr(kNameSpaceID_None, nsAccessibilityAtoms::tabindex) 
> #ifndef MOZ_ACCESSIBILITY_ATK
>            ||
>            tag == nsAccessibilityAtoms::blockquote ||
>+           tag == nsAccessibilityAtoms::q ||
>            tag == nsAccessibilityAtoms::form ||

Can you keep these listed in alphabetical order?

Comment 3

[Aaron Leventhal]

> Can you keep these listed in alphabetical order?
I can, but I wanted the 2 kinds of quotes together.

I'll check it in alphabetically. 

Comment 4

[Aaron Leventhal]

Checking in nsAccessibilityService.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityService.cpp,v  <-- 
nsAccessibilityService.cpp
new revision: 1.124; previous revision: 1.123
done

Comment 5

[Firefox Bug Husbandry Bot]

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