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

Selectively remove the "Open in new tab" option based on link type. #23

Closed
maxime-rainville opened this issue Nov 24, 2020 · 2 comments
Closed
Assignees
Labels

Comments

@maxime-rainville
Copy link

maxime-rainville commented Nov 24, 2020

Story

As a CMS User I want the ability to get links to open in new tab where it makes sense

Acceptance criteria

  • The "Open in new tab" option is the last form field by default.
  • The "Open in new tab" option is removed from PhoneLink and EmailLink.

Designs

Link field - Overview

Note

All Link type currently have that option displayed, including some where it doesn't make sense like the Email Link type.

The placement of the option is also weird, because it the second option all the time, which give it an undue prominence.

image

PR

@maxime-rainville
Copy link
Author

maxime-rainville commented Nov 30, 2020

Paul says:

  • leave to user to decide by default.
  • always display it at the end

Developers who are implementing custom link type will be responsible for removing the checkbox if they think it's not relevant.

@maxime-rainville maxime-rainville changed the title Open in new tab option Selectively removed the "Open in new tab" option based on link type. Jan 22, 2024
@emteknetnz emteknetnz self-assigned this Jan 23, 2024
@emteknetnz emteknetnz removed their assignment Jan 24, 2024
@GuySartorelli
Copy link
Member

PR merged

@GuySartorelli GuySartorelli changed the title Selectively removed the "Open in new tab" option based on link type. Selectively remove the "Open in new tab" option based on link type. Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants