Skip to content

fix: toolbar overflow command error#787

Merged
edoversb merged 1 commit intodevelopfrom
toolbar-overflow-command-error
Aug 18, 2025
Merged

fix: toolbar overflow command error#787
edoversb merged 1 commit intodevelopfrom
toolbar-overflow-command-error

Conversation

@mattConnHarbour
Copy link
Copy Markdown
Contributor

We can have a null command for a toolbar item. https://github.com/Harbour-Enterprises/SuperDoc/blob/toolbar-overflow-command-error/packages/super-editor/src/components/toolbar/super-toolbar.js#L841

This command (toggleOverflow) does not exist, so omitting it prevents this console error.

Copy link
Copy Markdown
Contributor

@edoversb edoversb left a comment

Choose a reason for hiding this comment

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

LGTM

@edoversb edoversb merged commit 5650b2a into develop Aug 18, 2025
7 checks passed
@edoversb edoversb deleted the toolbar-overflow-command-error branch August 18, 2025 21:27
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.

2 participants