Ticket #8001 (closed Bug: fixed)

Opened 3 years ago

Last modified 3 years ago

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

8001.patch (1.1 KB) - added by garry.yao 3 years ago.

Change History

comment:1 Changed 3 years ago by damo

Is there any update to this ticket?

comment:2 Changed 3 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: ↓ 5 Changed 3 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 3 years ago by j.swiderski

  • Status changed from new to closed
  • Resolution set to invalid

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

  • Status changed from closed to reopened
  • Resolution invalid deleted

comment:7 Changed 3 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 3 years ago by garry.yao

comment:8 Changed 3 years ago by garry.yao

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

We'd not keeping it.

comment:9 Changed 3 years ago by fredck

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

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

comment:10 Changed 3 years ago by garry.yao

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

Fixed with [7202].

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