Skip to content

Update to work with the latest version of 'richtext'#7

Open
wr2f25 wants to merge 2 commits intomainfrom
fix-tinymce
Open

Update to work with the latest version of 'richtext'#7
wr2f25 wants to merge 2 commits intomainfrom
fix-tinymce

Conversation

@wr2f25
Copy link
Contributor

@wr2f25 wr2f25 commented Jul 10, 2025

The richtext ingredient now uses TinyMCE 7.9.1 (eprints/richtext#10) so this changes initTinyMCE_for_easy_pages to match the changes in the version bump.

The textpattern ingredient has been removed and I believe is just included by default now so I have just removed that.

The preview_with_css_branding is a modified version of preview so I have pulled the latest version of preview (as the previous version was for 4.6.1) and then re-applied the modifications (filter content with preview_substitutions and add a font style-sheet).

This also removes an extra use of jQuery that I missed when I removed the others in #6.

wr2f25 added 2 commits July 10, 2025 10:57
The `richtext` ingredient now uses TinyMCE 7.9.1
(eprints/richtext#10) so this changes
`initTinyMCE_for_easy_pages` to match the changes in the version bump.

The `textpattern` ingredient has been removed and I believe is just
included by default now so I have just removed that.

The `preview_with_css_branding` is a modified version of `preview` so I
have pulled the new version of `preview` (as the previous version was
for 4.6.1) and then re-applied the modifications (filter content with
`preview_substitutions` and add a font style-sheet).
I thought I had done this in #6 however I somehow missed this use of
`jQuery.ready` so I have now replaced that with a `DOMContentLoaded`
listener. This is particularly relevant now that it works with the
latest version of `richtext` (which doesn't use jQuery).
@wr2f25 wr2f25 self-assigned this Jul 10, 2025
@wr2f25 wr2f25 added the bug Something isn't working label Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant