Opened 7 years ago

Last modified 7 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.

  1. Open CKEditor with tabletools plugin.
  2. 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">&nbsp;</td>
    			<td>&nbsp;</td>
    		</tr>
    	</tbody>
    </table>
    
  3. 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

Change History (0)

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy