Opened 8 years ago

Closed 8 years ago

#6976 closed Bug (duplicate)

about override in styles

Reported by: yiminghe Owned by:
Priority: Normal Milestone:
Component: Core : Styles Version: 3.5.1
Keywords: Cc:

Description

steps: 1.open www.ckeditor.com/demo 2.switch to source mode,input html:

<p>
	12345678<font style="background-color: rgb(130, 130, 130);">xyzabcd</font>9012345678</p>

3.switch to wysiwyg mode and place cursor between 'y' and 'z', then click background button , select '自动'

  1. expected:

'xyzabcd' does not have background-color.

actual result :

unchanged

5.select '8xyzabcd9' ,then click background button ,select red color

  1. expected :

generate code :

<p>
	1234567<span style="background-color: rgb(255, 0, 0);">8xyzabcd9</span>012345678</p>

actual code (too bloated):

<p>
	1234567<span style="background-color: rgb(255, 0, 0);">8</span><font style="background-color: rgb(130, 130, 130);"><span style="background-color: rgb(255, 0, 0);">xyzabcd</span></font><span style="background-color: rgb(255, 0, 0);">9</span>012345678</p>

ps:in trunk ,result is error :

<p>
	1234567<span style="background-color: rgb(255, 0, 0);">8<font style="background-color: rgb(130, 130, 130);">xyzabcd</font>9</span>012345678</p>

ps2: i provide a patch for trunk, add consideration about override element's style

Attachments (1)

override.patch (8.4 KB) - added by yiminghe 8 years ago.

Download all attachments as: .zip

Change History (2)

Changed 8 years ago by yiminghe

Attachment: override.patch added

comment:1 Changed 8 years ago by Krzysztof Studnik

Keywords: override style background-color removed
Resolution: duplicate
Status: newclosed

DUP of #6977

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