Skip to content

feat(OnyxTextEditor): use OnyxFormElementV2 internally#5225

Open
larsrickert wants to merge 15 commits intomainfrom
larsrickert/5144-text-editor
Open

feat(OnyxTextEditor): use OnyxFormElementV2 internally#5225
larsrickert wants to merge 15 commits intomainfrom
larsrickert/5144-text-editor

Conversation

@larsrickert
Copy link
Copy Markdown
Collaborator

@larsrickert larsrickert commented Apr 2, 2026

Relates to #5144

  • use OnyxFormElementV2 internally
  • move toolbar code to separate component to not overload the main component too much
  • support new properties: loading, error showError, required and requiredMarker
  • fix toolbar inline padding to match and align with the input

Checklist

  • The added / edited code has been documented with JSDoc
  • If a new component is added, at least one Playwright screenshot test or functional test is added
  • A changeset is added with pnpm exec changeset add if your changes should be released as npm package (because they affect the library usage)
  • I have performed a self review of my code ("Files changed" tab in the pull request)

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 2, 2026

🦋 Changeset detected

Latest commit: 4fae879

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sit-onyx/tiptap Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@larsrickert larsrickert force-pushed the larsrickert/5144-text-editor branch from 1e7fe7e to 4fae879 Compare April 2, 2026 13:51
@larsrickert larsrickert marked this pull request as ready for review April 2, 2026 13:51
@larsrickert larsrickert requested a review from a team as a code owner April 2, 2026 13:51
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.

1 participant