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

feat: drag and drop across multi editors #4328

Closed
wants to merge 6 commits into from

Conversation

songispm
Copy link
Contributor

@songispm songispm commented Aug 13, 2023

Please describe your changes

Delete content when drop on another editor,

At the same time, it does not affect the normal drop event effect of the editor.

Work in nested nodes too.

Core change of code:

https://github.com/ueberdosis/tiptap/pull/4328/files#diff-6eddae5591c39b131ab5597e8ac574df71fafda96023c4573c45fd73ed066b9f

Related issues

#3133

Demo

https://deploy-preview-4328--tiptap-embed.netlify.app/preview/Experiments/MultipleEditors

or checkout this pull request

  1. npm run dev
  2. Visit http://localhost:3000/preview/Experiments/MultipleEditors
iShot_2023-08-14_00.48.28.mp4

@netlify
Copy link

netlify bot commented Aug 13, 2023

Deploy Preview for tiptap-embed ready!

Name Link
🔨 Latest commit 3d37470
🔍 Latest deploy log https://app.netlify.com/sites/tiptap-embed/deploys/65a770729694710008105aff
😎 Deploy Preview https://deploy-preview-4328--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 configuration.

@songispm
Copy link
Contributor Author

Another related pull request: #3429

packages/core/src/PasteRule.ts Outdated Show resolved Hide resolved
@colinramsay
Copy link

@songispm is this ready to merge given the change which was requested was made?

@songispm
Copy link
Contributor Author

yes, I think it's worked.

https://github.com/ueberdosis/tiptap/pull/4328/files

@colinramsay
Copy link

Great work! Is it straightforward to do this React? I see there’s currently only a Vue example. @bdbch if you are happy do you think this could get in to a release?

@crystalball808
Copy link

Would like to use this feature!

@nperez0111 nperez0111 force-pushed the develop branch 2 times, most recently from bcaef5c to 7e7ae19 Compare June 12, 2024 04:33
@bdbch bdbch deleted the branch ueberdosis:develop June 26, 2024 21:57
@bdbch bdbch closed this Jun 26, 2024
@bdbch bdbch reopened this Jun 27, 2024
@bdbch
Copy link
Member

bdbch commented Nov 30, 2024

Moved to #5893

@bdbch bdbch closed this Nov 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants