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

Add support for custom selection builders #4128

Conversation

benglewis
Copy link
Contributor

Please describe your changes

Add support for custom selection builders.

How did you accomplish your changes

Added new field to the extension options, type and passing along parameter to the y-prosemirror extension. Passing by default the existing default value to avoid breaking anything.

How have you tested your changes

Ran the existing collaboration demo locally with the changes applied. Verified that the cursor and selection is shown correctly.

How can we verify your changes

You can clone code and check that the collaboration cursor and selection is not broken.

Remarks

N/A

Checklist

  • The changes are not breaking the editor
  • [] Added tests where possible - N/A
  • Followed the guidelines
  • Fixed linting issues

@netlify
Copy link

netlify bot commented Jun 14, 2023

Deploy Preview for tiptap-embed ready!

Name Link
🔨 Latest commit 9a29bd8
🔍 Latest deploy log https://app.netlify.com/sites/tiptap-embed/deploys/64abce9749419a000868a72b
😎 Deploy Preview https://deploy-preview-4128--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.

@benglewis
Copy link
Contributor Author

@bdbch Can you please take a look?

Copy link
Contributor

@bdbch bdbch left a comment

Choose a reason for hiding this comment

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

nice one! could you add the new selectionRender option to the docs too? @benglewis

@benglewis
Copy link
Contributor Author

@bdbch I've added the property to the documentation. Hopefully I understood how to do it correctly.

@benglewis benglewis requested a review from bdbch July 10, 2023 11:57
@RubaXa
Copy link

RubaXa commented Jul 10, 2023

@bdbch please, merge and release this request 🙇🏻‍♂️

@benglewis
Copy link
Contributor Author

@bdbch Any updates regarding this merge? 🙏

@benglewis
Copy link
Contributor Author

Hi @svenadlung @bdbch ,

Is there any chance that you can please review and merge this PR? 🙏
We are getting to the stage where we may need to patch TipTap for the meantime with this fix for our use-case and I'd prefer for us not to have to patch it

@bdbch bdbch merged commit 7cf17d0 into ueberdosis:develop Aug 3, 2023
15 checks passed
@benglewis benglewis deleted the extension-collaboration-cursor-custom-cursor-builder branch August 3, 2023 13:11
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

3 participants