Skip to content

Upgrade TinyMCE to v8 - #1516

Merged
petschki merged 2 commits into
masterfrom
petschki_tinymce_license_key
Dec 18, 2025
Merged

Upgrade TinyMCE to v8#1516
petschki merged 2 commits into
masterfrom
petschki_tinymce_license_key

Conversation

@petschki

@petschki petschki commented Dec 17, 2025

Copy link
Copy Markdown
Member

and use TinyMCE license_key from controlpanel if available.

This is a non-breaking feature implementation for Plone 6.2 only. I will bump mockup version to 5.5.x for that...

This also needs plone/plonetheme.barceloneta#427 for the updated tinymce-ui-content.css

EDIT:
this will take effect if plone/plone.base#104 and plone/Products.CMFPlone#4247 are merged ... othertwise we fallback to "gpl"

@mauritsvanrees mauritsvanrees left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I don't notice a difference when using this, but I don't know what to look for.

@petschki

petschki commented Dec 18, 2025

Copy link
Copy Markdown
Member Author

If you provide anything else than "gpl" in the new tinymce license_key field (TinyMCE Settings -> Advanced) you should see this message in tiny:
Bildschirmfoto 2025-12-18 um 12 17 12

Adding a valid license key needs documentation though, because you have to provide the licensekeymanager tinymce plugin which you only get when buying a license (of course) ... I'll add something here accordingly https://6.docs.plone.org/classic-ui/tinymce-customization.html ... maybe we should mention that in the 6.2 upgrade documentation too?

@thet thet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@petschki
petschki merged commit 1d464cc into master Dec 18, 2025
3 checks passed
@petschki
petschki deleted the petschki_tinymce_license_key branch December 18, 2025 14:20
@petschki petschki mentioned this pull request Dec 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants