Opened 10 years ago

Closed 10 years ago

#3557 closed Bug (fixed)

Disabled toolbar buttons are indistinguishable from normal toolbar buttons in JAWS

Reported by: Martin Kou Owned by: Martin Kou
Priority: Normal Milestone: CKEditor 3.0
Component: Accessibility Version:
Keywords: Oracle Review+ Cc: senthil.kumaran@…

Description

To reproduce:

  1. Open replacebyclass.html in any browser.
  2. Put caret to first paragraph - the decrease indent button should be disabled.
  3. Press Alt+F10.
  4. Press Tab until the focus goes to the button right before the decrease indent button.
  5. Press Tab.
  6. The decrease indent button gets the focus, and the getting read by JAWS - there is no indication that the button is disabled.

Attachments (3)

3357.patch (698 bytes) - added by Martin Kou 10 years ago.
3557.patch (3.0 KB) - added by Martin Kou 10 years ago.
3557_2.patch (2.3 KB) - added by Martin Kou 10 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 Changed 10 years ago by Martin Kou

Summary: Disabled toolbar buttons are indistinguishable to normal toolbar buttons in JAWSDisabled toolbar buttons are indistinguishable from normal toolbar buttons in JAWS

comment:2 Changed 10 years ago by Martin Kou

Status: newassigned

Changed 10 years ago by Martin Kou

Attachment: 3357.patch added

comment:3 Changed 10 years ago by Martin Kou

Keywords: Review? added

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Sorry, but this patch has nothing to do with this issue.

Changed 10 years ago by Martin Kou

Attachment: 3557.patch added

comment:5 Changed 10 years ago by Martin Kou

Please ignore the previous patch.

comment:6 Changed 10 years ago by Martin Kou

Keywords: Review? added; Review- removed

comment:7 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

I'm not sure about it, but I assume that different languages may have different rules to write the "unavailable" text. As this is out of our knowledge, we should provide a generic solution that certainly fit all needs. It means that the lang.unavailable entry should look like this:

unavailable : '%1, Unavailable' 

Also, the search and replace tricks to set the title are a bit weird. Isn't this.title enough for that?

Changed 10 years ago by Martin Kou

Attachment: 3557_2.patch added

comment:8 Changed 10 years ago by Martin Kou

Keywords: Review? added; Review- removed

comment:9 Changed 10 years ago by Senthil

Cc: senthil.kumaran@… added
Keywords: Oracle added; Review? removed

comment:10 Changed 10 years ago by Tobiasz Cudnik

Keywords: Review? added

Restoring missed keyword.

comment:11 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:12 Changed 10 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [3639].

Click here for more info about our SVN system.

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