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: link toolbar show/hide 'Open in new tab' option #528

Merged

Conversation

paulirwin
Copy link
Contributor

This PR contains:

  • bugfix
  • feature
  • refactor
  • tests
  • documentation
  • others

Breaking Changes?

  • yes
  • no

Checklist

  • commit messages follow the Conventional Commits pattern
    • A feature commit message is prefixed "feat:"
    • A bugfix commit message is prefixed "fix:"
    • A document related commit is prefixed "docs:"
  • docs have been added / updated (for bug fixes / features)

Describe Your Changes

This adds a new option for the Link toolbar icon/popup to disable the "Open in new tab" option from showing in the UI, for i.e. a Markdown serializer where target is not supported. Specifying "link" as a ToolbarItem is still supported and the default is to show the "Open in new tab" option, but you can now pass an object instead to specify its options via { link: { showOpenInNewTab: false } }, similar to the heading toolbar item.

Does this PR affects any existing issues?

  • yes
  • no

This adds a new option for the Link toolbar icon/popup to disable the
"Open in new tab" option from showing in the UI. Specifying `"link"` as
a ToolbarItem is still supported, but you can now pass an object to
specify its options via `{ link: { showOpenInNewTab: false } }`, similar
to the `heading` toolbar item.
Copy link
Owner

@sibiraj-s sibiraj-s left a comment

Choose a reason for hiding this comment

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

Thank you.

@sibiraj-s sibiraj-s enabled auto-merge (squash) April 3, 2024 06:08
@sibiraj-s sibiraj-s merged commit aacc680 into sibiraj-s:master Apr 3, 2024
1 check passed
@sibiraj-s
Copy link
Owner

Published in v17.0.2

@paulirwin paulirwin deleted the feature/open-in-new-tab-option branch April 3, 2024 14:08
Copy link

github-actions bot commented May 5, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in the thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants