Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Tinymce to v5 #2670

Merged
merged 7 commits into from
Jan 2, 2024
Merged

Update Tinymce to v5 #2670

merged 7 commits into from
Jan 2, 2024

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jan 2, 2024

What is this pull request for?

Updates bundled Tinymce Editor to v5.10.9

Notable changes

Updates the theme and icon set to match latest UI changes.

Please follow https://www.tiny.cloud/docs/migration-from-4x/ if you have custom plugins installed.

Screenshots

Checklist

Sorry, something went wrong.

@tvdeyen tvdeyen added this to the 7.1 milestone Jan 2, 2024
@tvdeyen tvdeyen requested a review from a team January 2, 2024 12:20
Copy link
Contributor

@sascha-karnatz sascha-karnatz left a comment

Choose a reason for hiding this comment

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

The tinymce feature spec is broken, but it looks pretty neat.

Tinymce 5 uses the complete editor interface for min height.
We can remove the complex min height calculation now.

Raising the default min height to a more usable height.
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 8782 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 8840 lines exceeds the maximum allowed for the inline comments feature.

We need the matchMedia mock, because - once again -
jsdom has not implemented it (*sigh). Otherwise the spec
will not even load, because Tinymce 5 needs this standard
HTML API do be defined.

The intersection observer spec does not work anymore.
The feature itself works in the browser, but not in Jest specs.
Skipping for now.
We do not use jQueryUI anymore.
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 8841 lines exceeds the maximum allowed for the inline comments feature.

@tvdeyen tvdeyen merged commit c59ab3d into main Jan 2, 2024
@tvdeyen tvdeyen deleted the tinymce-5 branch January 2, 2024 20:43
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.

None yet

2 participants