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

chore: allow new ReactComponentContent components to be created #3782

Merged
merged 2 commits into from Feb 27, 2023

Conversation

bdbch
Copy link
Contributor

@bdbch bdbch commented Feb 23, 2023

Inside the ReactComponentContent node view demo there was no way to add new components to test cursor behavior. This adds this functionality by using Mod+Enter.

This is related to #3781 as it was used to test the functionality this way.

@bdbch bdbch self-assigned this Feb 23, 2023
@netlify
Copy link

netlify bot commented Feb 23, 2023

Deploy Preview for tiptap-embed ready!

Name Link
🔨 Latest commit c26b1be
🔍 Latest deploy log https://app.netlify.com/sites/tiptap-embed/deploys/63fcf70120b7f600085edec8
😎 Deploy Preview https://deploy-preview-3782--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.

Copy link
Contributor

@svenadlung svenadlung left a comment

Choose a reason for hiding this comment

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

Not changing cursor position will be fixed with #3781, correct?

Hitting Mod + Enter in the middle of content splits content and adds another node – is it the desired behaviour? Anyways, it's just for demonstration purpose.

@bdbch
Copy link
Contributor Author

bdbch commented Feb 27, 2023

Not changing cursor position will be fixed with #3781, correct?

Hitting Mod + Enter in the middle of content splits content and adds another node – is it the desired behaviour? Anyways, it's just for demonstration purpose.

Exactly. #3781 fixes the cursor issue. And yes the demo could be improved, but for now it's usable to test the cursor positioning issue.

@bdbch bdbch merged commit acf186d into main Feb 27, 2023
@bdbch bdbch deleted the demo/update-react-demo branch February 27, 2023 18:31
aliasliao pushed a commit to aliasliao/tiptap that referenced this pull request May 24, 2023
…rdosis#3782)

* chore: allow users to create new react nodeview editor content components

* style: remove trailing spaces
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.

None yet

2 participants