- Issue created by @brad.bulger
- 🇺🇸United States justcaldwell Austin, Texas
I'm a definite +1 for extending support to other elements — headings, in my case.
Not opposed to
<div>support at all, but work in that direction seems questionable right now as "core" support for divs and other container elements is blocked upstream (see #3418322 ✨ [upstream] [GHS] Allow CKEditor 5 support for containers: div, article, section, aside, etc. Postponed ).I haven't tested Div Manager myself, but the CKE5 version (3.0) is still alpha. Judging from the issue queue the alpha tag is apt.
- 🇺🇸United States justcaldwell Austin, Texas
MR 13 allows any valid `$text-container` element (e.g. headers) to be indented.
It also updates the CSS to remove the element targets (p, ol, ul). List element are indented by nesting a new list container, so the class is not used in that case anyway. I also changed the units from `em` to `rem` to ensure consistent indents where font-sizes might vary (again, e.g. headings).
Static patch attached.