Ticket #6010 (closed Bug: fixed)
Automatic Background Color is Incorrectly displayed as "Black" in UI when it is actually "White"
| Reported by: | satya | Owned by: | Saare |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5 |
| Component: | General | Version: | 3.0 |
| Keywords: | IBM | Cc: | damo,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
Change History
comment:2 in reply to: ↑ 1 Changed 3 years ago by satya
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:8 in reply to: ↑ 1 Changed 3 years ago by garry.yao
- Status changed from review to 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'.
comment:10 Changed 3 years ago by garry.yao
- Status changed from review to 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;
comment:12 Changed 3 years ago by garry.yao
- Status changed from review to review_passed
A very informative feature added!
comment:13 Changed 3 years ago by Saare
- Status changed from review_passed to closed
- Resolution set to fixed
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?