Opened 8 years ago
Last modified 8 years ago
#16971 closed New Feature
tabletools should also retain color from background containing more than color — at Initial Version
| 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
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](https://github.com/ckeditor/ckeditor-dev/pull/337), where Firefox would put much more values into CSS prop ta
Note: See
TracTickets for help on using
tickets.
