Opened 8 years ago

Closed 8 years ago

#4354 closed Bug (wontfix)

Using TAB key on toolbar focus disabled buttons

Reported by: Tobiasz Cudnik Owned by: Tobiasz Cudnik
Priority: Normal Milestone: CKEditor 3.1
Component: Accessibility Version:
Keywords: Confirmed Cc:

Description

Using TAB key (or SHIFT+TAB) on toolbar focus disabled buttons, while they shouldn't be focusable.

To reproduce:

  • Open editor sample
  • Switch to source area
  • Focus editing area
  • Press ALT+F10
  • Press TAB

Result: Save button have focus (disabled)

Expected: New Page button should have focus (first non-disabled)

Attachments (1)

4354.patch (2.8 KB) - added by Tobiasz Cudnik 8 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 8 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: newassigned

Changed 8 years ago by Tobiasz Cudnik

Attachment: 4354.patch added

comment:2 Changed 8 years ago by Tobiasz Cudnik

Keywords: Review? added

I've added some additional comments to while loops as they were quite not clear to me at first sight.

comment:3 Changed 8 years ago by Garry Yao

Keywords: Review+ added; Review? removed

It's good to have TC covering this kind of accessibility feature, I'm trying to add one while it seems hard, let's compensate it later after the testing API is more stable.

comment:4 Changed 8 years ago by Tobiasz Cudnik

Keywords: NeedsTest added
Resolution: fixed
Status: assignedclosed

Fixed with [4171].

comment:5 Changed 8 years ago by Garry Yao

Keywords: Review+ NeedsTest removed
Resolution: fixed
Status: closedreopened

After discussing with Fred, it's incorrect to jump over the disabled item, the item should be announced properly for better accessibility. Revert it with [4186].

comment:6 Changed 8 years ago by Garry Yao

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