Opened 10 years ago

Closed 9 years ago

Last modified 9 years ago

#3224 closed Bug (expired)

CKEditor 3.0 : Ok/Cancel buttons are not working in Dialogs

Reported by: Senthil Owned by:
Priority: Normal Milestone:
Component: General Version:
Keywords: Oracle Pending Cc: Senthil

Description

When the JAWS is on, tab through the dialog(ex: Link or Image), once the focus moved to the ok/cancel press enter. This action should close or display some message but nothing happends.

This is reproducible in the sample html file provided by you. Sample File Used: replacebyclass.html

Attachments (1)

3224.patch (9.6 KB) - added by Martin Kou 10 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 10 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

comment:2 Changed 10 years ago by Martin Kou

Ok. This is again due to JAWS's virtual cursor needing to be updated by the user (Insert+Esc). If the user presses Insert+Esc every time a dialog is opened and closed (i.e. to use a table dialog, user would have to open table dialog -> Insert+Esc -> Ok/Cancel -> Insert+Esc -> Continue using editor), then the buttons would always work correctly.

Turns out this is a common problem with JavaScript-based dynamic content in webpages and screenreaders. http://juicystudio.com/article/making-ajax-work-with-screen-readers.php

We cannot force JAWS's virtual cursor to be updated from JavaScript. But we can inform the user to update his virtual cursor by putting a very short duration (200ms is enough on my computer) focus into a hidden element containing some text to be announced with voice.

comment:3 Changed 10 years ago by Martin Kou

Keywords: Review? added

Added the screenreader plugin, which is a way to direct JAWS to read out some text just before the HTML content is updated.

comment:4 Changed 10 years ago by Martin Kou

Priority: NormalHigh

This is a critical issue for Oracle's integration effort, so raising it to high priority.

Changed 10 years ago by Martin Kou

Attachment: 3224.patch added

comment:5 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.0CKEditor 3.x
Priority: HighNormal

I think we should take another direction for this fix. As Martin described, this is not really broken. It's just a matter of having the virtual cursor refreshed. So, what we really need is a way to inform visual impaired users that they have to do that. So, user documentation should be enough for it.

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Pending added; Review? removed

We're not anymore having issues with this with CKEditor 3.2. Can you please confirm it?

comment:7 Changed 10 years ago by Garry Yao

Owner: Martin Kou deleted
Status: assignednew

comment:8 Changed 9 years ago by Frederico Caldeira Knabben

Resolution: expired
Status: newclosed

comment:9 Changed 9 years ago by Frederico Caldeira Knabben

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