Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trailing Block plugin fixes #3226

Merged
merged 2 commits into from
May 28, 2024

Conversation

mareksuscak
Copy link
Contributor

Description

See changesets.

We're in the process of migrating from Slate to Plate and due to us wanting to maintain backward compatibility between the two editor data formats, we have to customize most plugins to use custom element types. We did notice that the trailing block plugin is broken when trying to override the trailing block type which is already supported in the options but isn't working correctly because the trailing block is always defaulting to the default type configured on an editor level which doesn't seem to be intended behavior.

Copy link

changeset-bot bot commented May 28, 2024

🦋 Changeset detected

Latest commit: d73acaa

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

This PR includes changesets to release 2 packages
Name Type
@udecode/plate-trailing-block Patch
@udecode/plate Patch

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

Copy link

vercel bot commented May 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
plate ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 28, 2024 7:33pm

@mareksuscak mareksuscak changed the title Forward the custom type Trailing Block plugin fixes May 28, 2024
@zbeyens zbeyens enabled auto-merge May 28, 2024 19:52
@zbeyens zbeyens merged commit c275f09 into udecode:main May 28, 2024
6 checks passed
@mareksuscak mareksuscak deleted the trailing-block-plugin-fixes branch May 28, 2024 20:12
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