287566 - Report "n of m" positional descriptions for list items and radio buttons

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

Screen readers need to know and report the current position in choice lists,
whether in lists, radio groups, combo boxes, etc.

Comment 1

[Aaron Leventhal]

Attachment: Move code for "n of m" descriptions into generic MSAA class, which allows it to be used by anything, including DHTML widgets

Comment 2

[Darin Fisher]

Comment on attachment 178480 [details] [diff] [review]
Move code for "n of m" descriptions into generic MSAA class, which allows it to be used by anything, including DHTML widgets

>Index: accessible/src/msaa/nsAccessibleWrap.cpp

>+NS_IMETHODIMP nsAccessibleWrap::GetDescription(nsAString& aDescription)

so, this seems to be the only implementation of this method now.
does it still need to be virtual?


sr=darin

Comment 3

[Aaron Leventhal]

Still needs to be virtual, we're going to override it for images and html radio
buttons. We already do override it for html area and xul tree accessibles.

Comment 4

[Aaron Leventhal]

Checking in accessible/src/atk/Makefile.in;
/cvsroot/mozilla/accessible/src/atk/Makefile.in,v  <--  Makefile.in
new revision: 1.13; previous revision: 1.12
done
Removing accessible/src/atk/nsXULTabAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/atk/nsXULTabAccessibleWrap.h,v  <-- 
nsXULTabAccessibleWrap.h
new revision: delete; previous revision: 1.1
done
Checking in accessible/src/base/nsAccessibilityService.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityService.cpp,v  <-- 
nsAccessibilityService.cpp
new revision: 1.132; previous revision: 1.131
done
Checking in accessible/src/base/nsAccessible.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessible.cpp,v  <--  nsAccessible.cpp
new revision: 1.138; previous revision: 1.137
done
Checking in accessible/src/mac/Makefile.in;
/cvsroot/mozilla/accessible/src/mac/Makefile.in,v  <--  Makefile.in
new revision: 1.11; previous revision: 1.10
done
Removing accessible/src/mac/nsXULTabAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/mac/nsXULTabAccessibleWrap.h,v  <-- 
nsXULTabAccessibleWrap.h
new revision: delete; previous revision: 1.1
done
Checking in accessible/src/msaa/Makefile.in;
/cvsroot/mozilla/accessible/src/msaa/Makefile.in,v  <--  Makefile.in
new revision: 1.10; previous revision: 1.9
done
Checking in accessible/src/msaa/nsAccessibleWrap.cpp;
/cvsroot/mozilla/accessible/src/msaa/nsAccessibleWrap.cpp,v  <-- 
nsAccessibleWrap.cpp
new revision: 1.22; previous revision: 1.21
done
Checking in accessible/src/msaa/nsAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/msaa/nsAccessibleWrap.h,v  <--  nsAccessibleWrap.h
new revision: 1.7; previous revision: 1.6
done
Removing accessible/src/msaa/nsXULTabAccessibleWrap.cpp;
/cvsroot/mozilla/accessible/src/msaa/nsXULTabAccessibleWrap.cpp,v  <-- 
nsXULTabAccessibleWrap.cpp
new revision: delete; previous revision: 1.2
done
Removing accessible/src/msaa/nsXULTabAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/msaa/nsXULTabAccessibleWrap.h,v  <-- 
nsXULTabAccessibleWrap.h
new revision: delete; previous revision: 1.1
done
Checking in accessible/src/other/Makefile.in;
/cvsroot/mozilla/accessible/src/other/Makefile.in,v  <--  Makefile.in
new revision: 1.8; previous revision: 1.7
done
Removing accessible/src/other/nsXULTabAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/other/nsXULTabAccessibleWrap.h,v  <-- 
nsXULTabAccessibleWrap.h
new revision: delete; previous revision: 1.1
done
Checking in accessible/src/xul/nsXULFormControlAccessible.cpp;
/cvsroot/mozilla/accessible/src/xul/nsXULFormControlAccessible.cpp,v  <-- 
nsXULFormControlAccessible.cpp
new revision: 1.45; previous revision: 1.44
done
Checking in accessible/src/xul/nsXULFormControlAccessible.h;
/cvsroot/mozilla/accessible/src/xul/nsXULFormControlAccessible.h,v  <-- 
nsXULFormControlAccessible.h
new revision: 1.23; previous revision: 1.22
done

Comment 5

[Firefox Bug Husbandry Bot]

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