#16971 closed New Feature (fixed)
tabletools should extract background color even for complex background value
| Reported by: | Marek Lewandowski | Owned by: | |
|---|---|---|---|
| Priority: | Nice to have (we want to work on it) | Milestone: | CKEditor 4.7.0 |
| Component: | General | Version: | |
| Keywords: | Cc: |
Description (last modified by )
Currently if td's background background property contains something more than just a color, it won't be recognized. We could improve it.
- Open CKEditor with tabletools plugin.
- Set the editor content using "Source" button to the following:
<table border="1" cellpadding="1" cellspacing="1" style="width:500px"> <tbody> <tr> <td style="background:#00cc99 no-repeat center"> </td> <td> </td> </tr> </tbody> </table>
- Go back to wysiwyg mode.
Expected
First cell has a backgournd applied.
Actual
Color is stripped out by ACF.
Additional Notes
Recently we have added a more advanced function for parsing CSS background - CKEDITOR.tools.style.parse.background. We could use it to more reliably tell if property contains a color.
Rationale: It was needed during PR#337, where Firefox would put much more values into CSS prop ta
Change History (4)
comment:1 Changed 8 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 8 years ago by
| Description: | modified (diff) |
|---|
comment:3 Changed 8 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | confirmed → closed |
comment:4 Changed 8 years ago by
| Description: | modified (diff) |
|---|---|
| Summary: | tabletools should also retain color from background containing more than color → tabletools should extract background color even for complex background value |
Note: See
TracTickets for help on using
tickets.
Merged to
majorwith 84d3983.