Opened 10 years ago

Closed 10 years ago

#5124 closed Bug (fixed)

Background colours cannot be set separately for spaces in a text string

Reported by: Senthil Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.3
Component: Core : Styles Version: SVN (CKEditor) - OLD
Keywords: Oracle Confirmed Review+ Cc: Senthil, Pranav

Description

If the space is having one character space between two text strings, the space could not get background color. If the space is having two or more characters space, The space get background color. Please see the attached screenshot.

Attachments (2)

BG Color issue.JPG (9.8 KB) - added by Senthil 10 years ago.
5124.patch (1.1 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by Senthil

Attachment: BG Color issue.JPG added

Changed 10 years ago by Garry Yao

Attachment: 5124.patch added

comment:1 Changed 10 years ago by Garry Yao

Component: GeneralCore : Styles
Keywords: Confirmed Review? added
Milestone: CKEditor 3.3
Owner: set to Garry Yao
Status: newassigned
Version: 3.1SVN (CKEditor)

This is definitely a bug.
Proposing of remove it unless somebody tell me 'hasContent' is necessary for a specific case.

comment:2 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review+ added; Review? removed

I guess it was an optimization to avoid setting "useless" styles on whitespace nodes like text-color, font-size... but there are other situations where such nodes can be important like setting IDs, or other kind of styles.

comment:3 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5227].

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