Closed
Bug 689847
Opened 13 years ago
Closed 13 years ago
Expose active state on current item of selectable widgets
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla11
People
(Reporter: surkov, Assigned: surkov)
References
(Blocks 2 open bugs)
Details
(Keywords: access)
Attachments
(1 file, 1 obsolete file)
7.31 KB,
patch
|
davidb
:
review+
|
Details | Diff | Splinter Review |
This is requirement of ARIA spec - http://www.w3.org/TR/wai-aria-implementation/#keyboard-focus_aria-activedescendant
If ARIA widgets needs that native widgets should follow to be consistent.
IA2 provides IA2_STATE_ACTIVE which is described as:
This state can be used to indicate the current active item in a container, even
if the container itself is not currently active. In other words this would indicate
the item that will get focus if you tab to the container.
Assignee | ||
Comment 1•13 years ago
|
||
besides we follow the spec this state should be useful for virtual cursor implementations when AT navigates the document without focus changing.
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attachment #577231 -
Flags: review?(bolterbugz)
Assignee | ||
Comment 2•13 years ago
|
||
actually ARIA part was implemented in bug 403260. Make items of all widgets to expose active state.
Attachment #577231 -
Attachment is obsolete: true
Attachment #577231 -
Flags: review?(bolterbugz)
Attachment #577236 -
Flags: review?
Assignee | ||
Updated•13 years ago
|
Attachment #577236 -
Flags: review? → review?(bolterbugz)
Comment 3•13 years ago
|
||
Comment on attachment 577236 [details] [diff] [review]
patch2
Review of attachment 577236 [details] [diff] [review]:
-----------------------------------------------------------------
r=me. nice.
::: accessible/src/base/nsAccessible.cpp
@@ +1548,5 @@
> + // descendant is the current one that would get focus if the user navigates
> + // to the container widget.
> + nsAccessible* widget = ContainerWidget();
> + if (widget && widget->CurrentItem() == this)
> + state |= states::ACTIVE;
Very nice change (I forgot about CurrentItem).
Attachment #577236 -
Flags: review?(bolterbugz) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Comment 5•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
Assignee | ||
Updated•13 years ago
|
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•