Opened 7 years ago

Closed 7 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 7 years ago.
5124.patch (1.1 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by Senthil

Attachment: BG Color issue.JPG added

Changed 7 years ago by Garry Yao

Attachment: 5124.patch added

comment:1 Changed 7 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 7 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 7 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5227].

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