Opened 6 years ago

Closed 5 years ago

#8523 closed Bug (fixed)

Changing colors not working with XHTML compliant code

Reported by: Martin Bauer Owned by:
Priority: Normal Milestone: CKEditor 3.6.3
Component: General Version: 3.2.1
Keywords: Cc:

Description

It is not possible to change the color twice, when using CKEditor in XHTML compliant mode. The bug can easily be reproduced on the demo site: http://nightly.ckeditor.com/7310/_samples/output_xhtml.html

Mark a word. Choose the blue color, which will work. Choosing a second color without unmarking the word will fail. This also makes it impossible to remove a chosen color.

Change History (6)

comment:1 Changed 6 years ago by Martin Bauer

Keywords: XHTML added

comment:2 Changed 6 years ago by Jakub Ś

Keywords: XHTML removed
Status: newconfirmed
Version: 3.2.1

It is not possible to change the color twice

  1. This is true for Text color. Once the color is set it cannot be changed. Looks like new span tags wrap old ones.
  2. This is not true for text background color as every time you choose color new span tag with class wraps selected text and not old tags.

This also makes it impossible to remove a chosen color

  1. This is true for text and text background color. Choosing twice the same color results in:
    1. span around text for font color <span class="FontColor1">some</span>
    2. two spans around text for text background color <span class="FontColor1BG"><span class="FontColor1BG">some</span></span>

(Switching to Source and back and then applying colors can help to reproduce the issue)

  1. It is also not possible to remove/change colors using "Automatic color"
Last edited 6 years ago by Jakub Ś (previous) (diff)

comment:3 Changed 6 years ago by Jakub Ś

Tickets #6749 and #8679 are related.

comment:4 Changed 6 years ago by Jakub Ś

#5960 is also related.

comment:5 Changed 6 years ago by Jakub Ś

Other tickets that fall under Select "Color Dialog Problems / Enchancements":
#8706, #8698, #8519, #8679, #8701, #8218, #5960, #4994.

comment:6 Changed 5 years ago by Alfonso Martínez de Lizarrondo

Milestone: CKEditor 3.6.3
Resolution: fixed
Status: confirmedclosed

This is fixed now with #7492.

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