Opened 12 years ago

Closed 12 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 12 years ago by Martin Bauer

Keywords: XHTML added

comment:2 Changed 12 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 12 years ago by Jakub Ś (previous) (diff)

comment:3 Changed 12 years ago by Jakub Ś

Tickets #6749 and #8679 are related.

comment:4 Changed 12 years ago by Jakub Ś

#5960 is also related.

comment:5 Changed 12 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 12 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 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy