open-formulieren / open-forms

Smart and dynamic forms
https://open-forms.readthedocs.io
Other
34 stars 25 forks source link

Certain WYSIWYG classes/options missing in front-end #4603

Open LaurensBurger opened 3 weeks ago

LaurensBurger commented 3 weeks ago

Product versie / Product version

2.6.13 - latest

Customer reference

lv 184

Omschrijf het probleem / Describe the bug

Examples are blockquotes, and text size: settings these either has no visible impact in the front-end or defaults to "user agent stylesheet" image image Where "text-big" does not exist in the style sheets.

Shouldn't these exist in the style sheet by default since they are available in the editor?

sergei-maertens commented 3 weeks ago

In short: no - we don't use the WYSIWYG editor stylesheets at all since they conflict too much. Most of the styles in these editors are done via inline-styles, this .text-big class is new to me.

Which editor is this - the formio WYSIWYG, or our own in the admin interface?

LaurensBurger commented 3 weeks ago

From our own admin interface.

sergei-maertens commented 3 weeks ago

Refinement: could be related to the TinyMCE upgrade, perhaps some inline styles have been replaced with class names? We need to check/confirm this, and check if we deliberately allow(ed) the blockquote content element.

If this should be supported, we need to update the CSS in the SDK to make sure it is displayed properly. NL DS components/styles may be relevant here, but let's keep it simple if possible.

sergei-maertens commented 3 weeks ago

@LaurensBurger how important is this, what's the prio?

LaurensBurger commented 3 weeks ago

@sergei-maertens low. Workaround(s) available.