Opened 9 years ago

Last modified 9 years ago

#13828 closed Bug

Widget classes should be added to wrapper rather than widget element — at Initial Version

Reported by: Marek Lewandowski Owned by:
Priority: Normal Milestone: CKEditor 4.6.0
Component: General Version:
Keywords: Cc:

Description

as @Reinmar writes:

Currently widget classes are applied to widget elements. This makes it impossible to achieve various results - e.g. ​http://stackoverflow.com/questions/29704019/how-to-floatleft-a-container-with-an-image-having-a-relative-width

We wanted to keep classes on elements because we thought that it allows to create the same stylesheets for the content inside editor and outside. Unfortunately, due to the wrappers and CSS's limited capabilities it makes it impossible to achieve some pretty standard results.

Note, that it will not be that uncommon to move classes to the wrapper because e.g. image2 already moves alignment classes there. It's a pity that we didn't learn from that fact earlier ;/.

So all the methods like addClass should add to the wrapper rather than to the widget's element.

This is part of #13200 issue.

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