Opened 6 years ago

Closed 5 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 6 years ago.
5124.patch (1.1 KB) - added by garry.yao 5 years ago.

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by Senthil

Changed 5 years ago by garry.yao

comment:1 Changed 5 years ago by garry.yao

  • Component changed from General to Core : Styles
  • Keywords Confirmed Review? added
  • Milestone set to CKEditor 3.3
  • Owner set to garry.yao
  • Status changed from new to assigned
  • Version changed from 3.1 to SVN (CKEditor)

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

comment:2 Changed 5 years ago by alfonsoml

  • 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 5 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [5227].

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