Skip to content

feat(tiptap): add Tiptap toolbar support#2793

Open
tammi-23 wants to merge 1 commit into
mainfrom
feat/tiptap-toolbars
Open

feat(tiptap): add Tiptap toolbar support#2793
tammi-23 wants to merge 1 commit into
mainfrom
feat/tiptap-toolbars

Conversation

@tammi-23

@tammi-23 tammi-23 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

This PR adds toolbar variants to the shared Tiptab text editor. The initial usecase is supporting different toolbar layouts for different mail compose sizes.

@JammingBen JammingBen left a comment

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'm not the biggest fan of this solution because it seems to be heavily tailored to the mail app. What if some other app wants to display a different set of toolbar items on mobile?

I'd rather have a more generalized mechanism of controlling which actions to display. Maybe introduce a actionsToDisplay prop on the toolbar which acts as a whitelist and defaults to undefined = all actions? The downside would be that the consuming party would require knowledge about the action ids. But it would keep the whole thing more abstract and not tailored to a specific implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants