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

fix(core): allow insertContentAt and insertContent text node arrays #3790

Merged
merged 2 commits into from Feb 27, 2023

Conversation

bdbch
Copy link
Contributor

@bdbch bdbch commented Feb 24, 2023

This PR fixes #2770 by checking the inserted text content if it's of type array. If so we can be sure that the content will be an array of text content.

@bdbch bdbch self-assigned this Feb 24, 2023
@bdbch bdbch added this to the 2.0.0 Release milestone Feb 24, 2023
@netlify
Copy link

netlify bot commented Feb 24, 2023

Deploy Preview for tiptap-embed ready!

Name Link
🔨 Latest commit efe15f1
🔍 Latest deploy log https://app.netlify.com/sites/tiptap-embed/deploys/63f94b2aa230370008f482b2
😎 Deploy Preview https://deploy-preview-3790--tiptap-embed.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@svenadlung
Copy link
Contributor

Thx @bdbch 🎉

@bdbch bdbch merged commit 0300630 into main Feb 27, 2023
@bdbch bdbch deleted the fix/allow-insert-content-with-json branch February 27, 2023 18:26
aliasliao pushed a commit to aliasliao/tiptap that referenced this pull request May 24, 2023
…eberdosis#3790)

* fix(core): allow insertContentAt and insertContent to handle array of text nodes

* fix(core): allow insertContent via json including a text content
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.

Problem when using insertContent with a text node list
2 participants