Skip to content

DOC-2898: Documentation enhancements: for https://www.tiny.cloud/docs/tinymce/latest/custom-toggle-toolbar-button/#4071

Merged
kemister85 merged 9 commits into
tinymce/8from
hotfix/8/testingPR
Jun 4, 2026
Merged

DOC-2898: Documentation enhancements: for https://www.tiny.cloud/docs/tinymce/latest/custom-toggle-toolbar-button/#4071
kemister85 merged 9 commits into
tinymce/8from
hotfix/8/testingPR

Conversation

@jeslynbo
Copy link
Copy Markdown

@jeslynbo jeslynbo commented Apr 16, 2026

Ticket: DOC-2898

Site: Staging branch

Changes:

  • Simplified onSetup event handling and teardown logic
  • Replacing plain inline code with a cross-referenced API link
  • Edited comments for onSetup demo
  • Ensure {product name} is being used correctly

Pre-checks:

  • Branch prefixed with feature/<version>/, hotfix/<version>/, staging/<version>/, or release/<version>/.
  • modules/ROOT/nav.adoc has been updated (if applicable).
  • Included a release note entry for any New product features.
  • If this is a minor release, updated productminorversion in antora.yml and added new supported versions entry in modules/ROOT/partials/misc/supported-versions.adoc.

Review:

  • Documentation Team Lead has reviewed

@jeslynbo jeslynbo requested review from a team, kemister85 and soritaheng as code owners April 16, 2026 01:37
@jeslynbo jeslynbo marked this pull request as draft April 16, 2026 01:40
…criptions for component creation and destruction callbacks.
@jeslynbo jeslynbo marked this pull request as ready for review April 20, 2026 09:42
Copy link
Copy Markdown
Contributor

@kemister85 kemister85 left a comment

Choose a reason for hiding this comment

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

Good improvements overall — the restructured explanation, clearer option/API descriptions, and the formatChanged documentation with unbind() are all valuable. A few issues to fix before merge (suggestions provided inline so you can commit them directly).

Comment thread modules/ROOT/pages/custom-toggle-toolbar-button.adoc Outdated
Comment thread modules/ROOT/pages/custom-toggle-toolbar-button.adoc Outdated
Comment thread modules/ROOT/pages/custom-toggle-toolbar-button.adoc Outdated
Comment thread modules/ROOT/pages/custom-toggle-toolbar-button.adoc Outdated
Comment thread modules/ROOT/pages/custom-toggle-toolbar-button.adoc Outdated
Comment thread modules/ROOT/pages/custom-toggle-toolbar-button.adoc Outdated
Comment thread modules/ROOT/partials/misc/onSetup.adoc Outdated
jeslynbo and others added 6 commits May 11, 2026 11:29
Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
Co-authored-by: Karl Kemister-Sheppard <karlkemistersheppard@gmail.com>
@kemister85 kemister85 self-requested a review June 1, 2026 22:53
Copy link
Copy Markdown
Contributor

@kemister85 kemister85 left a comment

Choose a reason for hiding this comment

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

LGTM

@kemister85 kemister85 merged commit 47d1880 into tinymce/8 Jun 4, 2026
5 checks passed
@kemister85 kemister85 deleted the hotfix/8/testingPR branch June 4, 2026 03:23
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