Opened 6 years ago

Closed 6 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)

8001.patch (1.1 KB) - added by Garry Yao 6 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 6 years ago by Damian

Is there any update to this ticket?

comment:2 Changed 6 years ago by Jakub Ś

Status: newconfirmed
Version: 3.2

In case of CKEditor dropdowns selection doesn’t follow focus. aria-activedescendant is only updated on selection.

comment:3 Changed 6 years ago by Garry Yao

Status: confirmednew

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 6 years ago by Jakub Ś

Resolution: invalid
Status: newclosed

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 in reply to:  3 Changed 6 years ago by Damian

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 6 years ago by Jakub Ś

Resolution: invalid
Status: closedreopened

comment:7 Changed 6 years ago by Jakub Ś

Status: reopenedconfirmed

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 6 years ago by Garry Yao

Attachment: 8001.patch added

comment:8 Changed 6 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

We'd not keeping it.

comment:9 Changed 6 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.2
Status: reviewreview_passed

If we have the option to make it simple, let's go ahead with it.

comment:10 Changed 6 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7202].

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy