Opened 8 years ago

Last modified 4 years ago

#8679 closed Bug

Focus is not indicated clearly when navigating the color picker dialog — at Version 1

Reported by: Teresa Monahan Owned by:
Priority: Normal Milestone: CKEditor 4.5.9
Component: General Version: 3.0
Keywords: IBM Cc: Damian, Satya Minnekanti

Description (last modified by Jakub Ś)

Load any CKEditor sample and open the color picker dialog by clicking the 'More Colors..' option on the 'Text Color' or 'Background Color' toolbar icons for example. Use the keyboard to navigate the color table.

Problem: The focus indicator around the currently selected color is not very clear.

CKEditor relies on the browser to display visual focus, however this can be difficult to see against the different background colors. In fact the focus is not displayed at all in recent versions of FF.

We would like to be able to add our own styling to focused elements on the color picker dialog.This is already possible for the special characters dialog through the cke_light_background class. We would like a similar capability for the color picker dialog.

For example a CSS class could be added to the current selection on the color table in updateHighlight() and updateSelected() in colordialog/dialogs/colordialog.js. This class would then need to be removed when another color is selected.

As this is an accessibility concern, this is a high priority for us and we would appreciate a fix in a 3.6.x release.

Change History (1)

comment:1 Changed 8 years ago by Jakub Ś

Description: modified (diff)
Status: newconfirmed
Version: 3.6.3 (SVN - trunk)3.0

We would like to be able to add our own styling to focused elements on the color picker dialog. This is already possible for the special characters dialog through the cke_light_background class.

Custom styling of currently selected color is a good idea, however IMO before this can added we first have to deal with #6749 as those two are related.

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