Skip to content

Update demos to TinyMCE 8 - #38

Merged
lostkeys merged 6 commits into
masterfrom
hotfix/tinymce-8
Nov 19, 2025
Merged

lostkeys merged 6 commits into
masterfrom
hotfix/tinymce-8

Conversation

@MitchC1999

Copy link
Copy Markdown
Collaborator

Update demos to TinyMCE 8

@ShiridiGandham ShiridiGandham left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@MitchC1999

Copy link
Copy Markdown
Collaborator Author

Decision has been made to archive this repo instead.

@MitchC1999 MitchC1999 closed this Nov 18, 2025
@lostkeys lostkeys reopened this Nov 18, 2025
Comment thread cool-demos/expand-editor-and-show-toolbar.html
Comment thread cool-demos/slash-commands.html
Comment thread email/webmail-rich-text-editor.html

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.

The table row and column toolbar menu buttons do not work, throws an error

TypeError: editor.plugins.table.getClipboardRows is not a function. (In 'editor.plugins.table.getClipboardRows()', 'editor.plugins.table.getClipboardRows' is undefined)

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.

Insert a templated row (drone status) do not work and throws an error

TypeError: editor.plugins.table.getClipboardRows is not a function. (In 'editor.plugins.table.getClipboardRows()', 'editor.plugins.table.getClipboardRows' is undefined)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Would need a dev to advise on if this functionality is even possible with TinyMCE 8 for sure. I can't find a way to do it after looking at the table plugin's commands and APIs for a bit.

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.

I think it was an undocumented API, we replaced it with the "fake clipboard" concept (which doesn't have table-specific APIs but the table plugin uses it to store the same data).

Comment thread tinymce-basics/webapp-description-comment-reply-editor.html
Comment thread tinymce-basics/webapp-description-comment-reply-editor.html
Comment thread tokens/span-based-token.html Outdated
Comment thread tokens/text-based-token.html Outdated
Comment thread tokens/web-component-token.html Outdated
tinymce-basics/useful_styles_for_content_css.html
* Fixed invalid css
* Re-added "hr" in the content
* Removed `noneditable` plugin
* Updated all documentation links

tinymce-basics/badges.html
* Removed `noneditable` plugin
* Updated all documentation links

tinymce-basics/custom-toolbar-button-menu-and-context-toolbar.html
* Removed `noneditable` plugin
* Updated all documentation links

tinymce-basics/webapp-description-comment-reply-editor.html
* Correctly use text pattern feature
* Removed text pattern plugin
* Updated all documentation links
* Added markdown plugin

tinymce-basics/bootstrap-rich-text-form.html
* Updated all documentation links

tinymce-basics/right-aligned-toolbar-button.html
* Removed demo because we now have various badges showing up here

tinymce-basics/format-link-as-button.html
* Updated all documentation links

tinymce-basics/fonts.html
* Updated all documentation links
* replaced font_formats with font_family_formats to make custom fonts show up

images/toggle-image-size-context-toolbar.html
* Updated all documentation links

images/image-alt-text-context-form.html
* Updated all documentation links

tables/customizing-the-table-toolbar.html
* Updated all documentation links
* TODO: Commented out the cut/copy/paste row & column buttons preventing the menu from working, waiting for someone more talented to fix them using the new fake clipboard api: https://www.tiny.cloud/docs/tinymce/latest/apis/tinymce.fakeclipboard/

tables/table-row-templates.html
* TODO: someone needs to update the copy paste row using the new fake clipboard api: https://www.tiny.cloud/docs/tinymce/latest/apis/tinymce.fakeclipboard/ otherwise the whole premise of the demo is gone
* Updated all documentation links

templates-and-layout/advanced-placeholder.html
* Removed demo because the new enhanced templates plugin does not have the same features as the template plugin it replaced, and cannot be used with this demo, therefore the premise of the demo is lost.

templates-and-layout/multiple-columns-using-css-columns.html
* Removed demo because using css this way is not something TinyMCE supports very well, and I feel it is unwise to lead someone down this route.

templates-and-layout/locked-down-template-inline-default.html
* Updated all documentation links

templates-and-layout/locked-down-template-inline-context-toolbars.html
* Updated all documentation links

templates-and-layout/locked-down-template-inline-context-toolbars.html
* Updated all documentation links

templates-and-layout/locked-down-template-inline-bordered.html
* Updated all documentation links
* Re-added the borderless skin to fix broken toolbar visuals

email/webmail-rich-text-editor.html
* Switched to naked skin because the outside skin had the editor outline on top of the bottom toolbar
* Updated all documentation links

email/marketing-email-inline-mode.html
* Updated all documentation links

email/marketing-email-inline-mode-fixed-toolbar.html
* Updated all documentation links

email/marketing-email.html
* Updated the demo to use the mergetags plugin
* Updated all documentation links

cool-demos/expand-editor-and-show-toolbar.html
* Fixed invalid CSS
* Updated all documentation links
* Note that toolbar bottom with the new tooltips makes it almost impossible to use the toolbar

cool-demos/conditional-blocks.html
* Updated all documentation links
* Removed `noneditable` plugin

cool-demos/slash-commands.html
* Added the `tableofcontents` plugin
* Updated all documentation links
Comment thread cool-demos/conditional-blocks.html Outdated
Comment thread cool-demos/conditional-blocks.html Outdated
Comment thread cool-demos/conditional-blocks.html Outdated
Comment thread cool-demos/conditional-blocks.html Outdated
Comment thread cool-demos/expand-editor-and-show-toolbar.html
// TinyMCE 5.4 introduced improved APIs for inserting tables.
// TinyMCE has APIs for inserting tables.
// Here's a toolbar button that displays a table grid picker
// The table picker is unfortunately undocumented at this stage

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Feels weird to just say this. Is this something we want to document? Looks like we don't document fancy menu items at all. "At this stage" implies that we might document it down the track...

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Alternatively - why do we have an "inserttable" menu item but not a toolbar button in TinyMCE? Should we add one?? Questions for outside the scope of this PR.

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.

It's a question that comes up occasionally from customers as well. I sometimes regret my inability to put us on a good architecture with TinyMCE 5 that would have made both work automatically.

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.

I left it as is. Can we simply add it to the docs?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Probably worth documenting it at least, if customers are asking. I'll flag it with Karl

Comment thread tables/customizing-the-table-toolbar.html Outdated
Comment thread tables/customizing-the-table-toolbar.html Outdated
Comment thread tables/customizing-the-table-toolbar.html Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Would need a dev to advise on if this functionality is even possible with TinyMCE 8 for sure. I can't find a way to do it after looking at the table plugin's commands and APIs for a bit.

@lostkeys
lostkeys self-requested a review November 19, 2025 13:43
@lostkeys
lostkeys merged commit e9a2ded into master Nov 19, 2025
3 checks passed
@lostkeys
lostkeys deleted the hotfix/tinymce-8 branch November 19, 2025 13:47
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.

8 participants