- Issue created by @kopeboy
- First commit to issue fork.
- Merge request !163472777 119 Added elements used by todo list to allowed for Limit allowed HTML tags filter → (Open) created by salmonek
- 🇵🇱Poland salmonek
Hi @kopeboy
Thank you for bringing this oversight to our attention.
Posted patch adds tags required by todo list to the "Allowed HTML tags" list. - 🇮🇹Italy kopeboy Milan
Since we are here (😅😇).. I noticed that the relevant CSS is not applied to the front-end theme, ie. when there is no CKEditor active on the page..
so shouldn't we remove the
.ck-contentclass prefix from this CSS below, to make it look good out-of-the-box everywhere?.ck-content .todo-list .todo-list__label .todo-list__label__description { vertical-align: middle; }.ck-content .todo-list { list-style: none; }The margins top & bottom instead are irrelevant as a frontend theme will always have them already on lists (so I wouldn't touch all other CSS), but the margin-left is needed and missing from the frontend (see below), so I would add
margin-left: 1ch;to the first declaration above (without the.ck-contentclass prefix).AS IS while editing (with Claro default theme):
AS IS while viewing (with Olivero default theme):
TO BE while viewing (with Olivero default theme and the edits I described):