Opened 14 years ago
Closed 14 years ago
#6010 closed Bug (fixed)
Automatic Background Color is Incorrectly displayed as "Black" in UI when it is actually "White"
| Reported by: | Satya Minnekanti | Owned by: | Sa'ar Zac Elias |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5 |
| Component: | General | Version: | 3.0 |
| Keywords: | IBM | Cc: | Damian, joek |
Description
To reproduce the defect
- Open Ajax sample.
- Enter some text in the Editor,Then click "Background color" menu,select the "Automatic" color.
Problem:
The Automatic color of Background color menu marks black,but it is actually white.
Attachments (3)
Change History (16)
comment:1 follow-ups: 2 8 Changed 14 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 14 years ago by
Replying to fredck:
We can try using the document <body> color instead of the black square. Would that be enough?
I think that should be OK
comment:3 Changed 14 years ago by
| Milestone: | → CKEditor 3.5 |
|---|
comment:4 Changed 14 years ago by
| Owner: | set to Sa'ar Zac Elias |
|---|---|
| Status: | confirmed → assigned |
Changed 14 years ago by
| Attachment: | 6010.patch added |
|---|
comment:5 Changed 14 years ago by
| Status: | assigned → review |
|---|
comment:6 Changed 14 years ago by
| Version: | 3.3 → 3.0 |
|---|
comment:7 Changed 14 years ago by
| Milestone: | CKEditor 3.4.1 → CKEditor 3.5 |
|---|
comment:8 Changed 14 years ago by
| Status: | review → review_failed |
|---|
Replying to fredck:
We can try using the document <body> color instead of the black square. Would that be enough?
It's counter-intuitive to fill with <body> color as the semantic of 'auto' color should be explained instead as 'computed color style of the current element'.
Changed 14 years ago by
| Attachment: | 6010_1.patch added |
|---|
comment:9 Changed 14 years ago by
| Status: | review_failed → review |
|---|
comment:10 Changed 14 years ago by
| Status: | review → review_failed |
|---|
Fails with the following content when open panel on 'cell2'.
<table style="width: 500px; background-color: #dcdcdc;"> <tbody> <tr> <td> cell1</td> <td bgcolor="#9acd32"> cell2</td> </tr> </tbody> </table>
Please simply use the following to check blocks:
var path = new CKEDITOR.dom.elementPath( editor.getSelection().getStartElement() ); block = path.block || path.blockLimit;
Changed 14 years ago by
| Attachment: | 6010_2.patch added |
|---|
comment:11 Changed 14 years ago by
| Status: | review_failed → review |
|---|
comment:13 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5935].
Note: See
TracTickets for help on using
tickets.
We can try using the document <body> color instead of the black square. Would that be enough?