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

extension/link: adds 'whenNotEditable' as option for openOnClick #3312

Merged
merged 7 commits into from
Apr 8, 2024
Merged

extension/link: adds 'whenNotEditable' as option for openOnClick #3312

merged 7 commits into from
Apr 8, 2024

Conversation

benkroeger
Copy link
Contributor

@benkroeger benkroeger commented Oct 14, 2022

Allows 'openOnClick' configuration to be set to 'whenNotEditable' so that links are not clickable while editing the content.
I have noticed that the editing experience is a little bumpy when you click on a 'link' marked text e.g. to modify the text - but then the link opens.

related issue: #4877

@netlify
Copy link

netlify bot commented Oct 14, 2022

Deploy Preview for tiptap-embed ready!

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

@benkroeger
Copy link
Contributor Author

@bdbch is there anything else I can do to move this forward?

@bdbch bdbch added Status: Review Category: Open Source The issue or pull reuqest is related to the open source packages of Tiptap. labels Mar 29, 2023
bdbch
bdbch previously approved these changes Jun 23, 2023
* upstream/main: (399 commits)
  added new labels
  update links.yaml
  improve GitHub issue templates
  improve GitHub issue templates
  v2.1.11
  Revert "v2.2.11"
  v2.2.11
  feature(core): pass through paste event to pasteHandler `getAttributes` (#4354)
  v2.1.10
  Fix React and Vue NodeView renderers not adding correct selection classes  (#4452)
  v.2.1.9
  chore: dont generate changelogs for rc versions
  fix: add missing attributes in extension-link (#4429)
  docs(react): fix typo on React installation code block (#4441)
  v2.1.8
  v2.1.7
  prevent opening link when clicking on selection containing a link (#4346)
  docs: complete hard break shortcuts
  docs: complete hard break shortcuts
  docs(collaboration): fix typo in collaboration.md
  ...

Signed-off-by: Benjamin Kroeger <benjamin.kroeger@gmail.com>

# Conflicts:
#	packages/extension-link/src/helpers/clickHandler.ts
#	packages/extension-link/src/link.ts
@bdbch
Copy link
Contributor

bdbch commented Oct 10, 2023

LGTM @svenadlung need a review from you so we can push this to RC

@benkroeger
Copy link
Contributor Author

@bdbch @svenadlung how can I help to move this forward?

@benkroeger
Copy link
Contributor Author

just merged the latest main branch into this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Open Source The issue or pull reuqest is related to the open source packages of Tiptap.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants