Opened 14 years ago
Closed 14 years ago
#5242 closed Bug (fixed)
Text and Background Colour applied to the text are removed together even though we removed only one of Text Colour or Background Colour
| Reported by: | Satya Minnekanti | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | Core : Styles | Version: | SVN (CKEditor) - OLD |
| Keywords: | IBM Confirmed Review+ | Cc: | Damian, joek |
Description
To reproduce the defect:
- open the Ajax sample.
- Click on Text Colour button menu and select some colour(eg: red).
- Click on Background Colour button menu and select some colour(eg: cyan).
- Type the text see that both the Text and Back ground colour's that we have selcted in steps 2 & 3 is applied to the text.
- Now click on Text Colour button and select the Automatic Colour Option to remove the Text Colour.
- Type some more text.
- See that both Text Colour as well as Background colour applied to the text is removed.
Expected Result:
Only the Text Colour should be removed and the Background colour should remain.
Same behaviour appears when we remove Background Colour insted of Text Colour in step 5.
Attachments (2)
Change History (10)
comment:1 Changed 14 years ago by
| Component: | General → Core : Styles |
|---|---|
| Keywords: | Confirmed added |
| Owner: | set to Garry Yao |
| Status: | new → assigned |
| Version: | 3.1 → SVN (CKEditor) |
Changed 14 years ago by
| Attachment: | 5242.patch added |
|---|
comment:2 Changed 14 years ago by
| Keywords: | Review? added |
|---|
comment:5 Changed 14 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
In IE6 and IE7 this is still re producible.
Changed 14 years ago by
| Attachment: | 5242_2.patch added |
|---|
comment:6 Changed 14 years ago by
| Keywords: | Review? added; Review+ removed |
|---|
New patch that targeting IE6/7.
comment:7 Changed 14 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Note: See
TracTickets for help on using
tickets.
The 'Automatic' color options works now by using a hack that an empty color style (single span without any attributes) could matches all fore and background colors, but obviously this's cumbersome, since it will incorrectly remove the one other when applying, so introduce here a new approach.