Opened 14 years ago
Closed 14 years ago
#8001 closed Bug (fixed)
aria-activedescendant incorrectly used in combos
| Reported by: | Damian | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.2 |
| Component: | Accessibility | Version: | 3.2 |
| Keywords: | IBM | Cc: | Satya Minnekanti, Teresa Monahan |
Description
This ticket is a continuation from #5403. It is possibly contributing to the problem in #5403. In any case, this ticket can be addressed in isolation from #5403.
The aria-activedescendant attribute is not being properly updated by the combos as the user changes the "currently active descendant" of the list. If aria-activedescendant is used, it should be used to indicate the currently focused item in the list. See http://www.w3.org/TR/wai-aria/states_and_properties#aria-activedescendant for more details.
Attachments (1)
Change History (11)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
| Status: | new → confirmed |
|---|---|
| Version: | → 3.2 |
In case of CKEditor dropdowns selection doesn’t follow focus. aria-activedescendant is only updated on selection.
comment:3 follow-up: 5 Changed 14 years ago by
| Status: | confirmed → new |
|---|
The spec clearly tells that when focus is managed by the app there's no need for the existence of this aria state at all which is exactly the case in the editor.
This is used when a composite widget is responsible for managing its current active child to reduce the overhead of having all children be focusable...
Besides, we've properly presented aria-selected on listbox items, instead it's a JAW issue for not making it in effect.
comment:4 Changed 14 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
Based on @garry.yao comment I'm closing the ticket.
@damo if you don't agree with @garry.yao argumentation you can always reopen this issue.
comment:5 Changed 14 years ago by
Replying to garry.yao:
... there's no need for the existence of this aria state at all which is exactly the case in the editor. ...
The whole problem is that the aria-activedescendant attribute is on the listbox and it isn't being updated by the application properly. So you have two options, start updating it properly or remove it entirely. It can't stay in there as it is right now as that is certainly wrong and introduces a level of uncertainty to the accessible APIs.
@j.swiderski please re-open.
comment:6 Changed 14 years ago by
| Resolution: | invalid |
|---|---|
| Status: | closed → reopened |
comment:7 Changed 14 years ago by
| Status: | reopened → confirmed |
|---|
Changing the status once again to confirmed.
Could I just ask you all to re-discus this issue first and than change the status to appropriate one:).
Changed 14 years ago by
| Attachment: | 8001.patch added |
|---|
comment:8 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
We'd not keeping it.
comment:9 Changed 14 years ago by
| Milestone: | → CKEditor 3.6.2 |
|---|---|
| Status: | review → review_passed |
If we have the option to make it simple, let's go ahead with it.
comment:10 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [7202].
Is there any update to this ticket?