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)
Change History (5)
Changed 7 years ago by
| Attachment: | BG Color issue.JPG added |
|---|
Changed 7 years ago by
| Attachment: | 5124.patch added |
|---|
comment:1 Changed 7 years ago by
| Component: | General → Core : Styles |
|---|---|
| Keywords: | Confirmed Review? added |
| Milestone: | → CKEditor 3.3 |
| Owner: | set to Garry Yao |
| Status: | new → assigned |
| Version: | 3.1 → SVN (CKEditor) |
comment:2 Changed 7 years ago by
| 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.
Note: See
TracTickets for help on using
tickets.
This is definitely a bug.
Proposing of remove it unless somebody tell me 'hasContent' is necessary for a specific case.