Ticket #5124 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

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

BG Color issue.JPG (9.8 KB) - added by Senthil 5 years ago.
5124.patch (1.1 KB) - added by garry.yao 5 years ago.

Change History

Changed 5 years ago by Senthil

Changed 5 years ago by garry.yao

comment:1 Changed 5 years ago by garry.yao

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

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

comment:2 Changed 4 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 4 years ago by garry.yao

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

Fixed with [5227].

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