Opened 4 years ago

Closed 4 years ago

#8001 closed Bug (fixed)

aria-activedescendant incorrectly used in combos

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.2
Component: Accessibility Version: 3.2
Keywords: IBM Cc: satya, tmonahan

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 4 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 4 years ago by damo

Is there any update to this ticket?

comment:2 Changed 4 years ago by j.swiderski

  • Status changed from new to confirmed
  • Version set to 3.2

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

comment:3 follow-up: Changed 4 years ago by garry.yao

  • Status changed from confirmed to 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 4 years ago by j.swiderski

  • Resolution set to invalid
  • Status changed from new to 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 in reply to: ↑ 3 Changed 4 years ago by damo

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 4 years ago by j.swiderski

  • Resolution invalid deleted
  • Status changed from closed to reopened

comment:7 Changed 4 years ago by j.swiderski

  • Status changed from reopened to 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 4 years ago by garry.yao

comment:8 Changed 4 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

We'd not keeping it.

comment:9 Changed 4 years ago by fredck

  • Milestone set to CKEditor 3.6.2
  • Status changed from review to review_passed

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

comment:10 Changed 4 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [7202].

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