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

CMD + Click on a link = opens the link in a new tab #1381

Closed
pomek opened this issue Nov 29, 2018 · 6 comments Β· Fixed by #10737
Closed

CMD + Click on a link = opens the link in a new tab #1381

pomek opened this issue Nov 29, 2018 · 6 comments Β· Fixed by #10737
Assignees
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:link squad:core Issue to be handled by the Core team. type:feature This issue reports a feature request (an idea for a new functionality or a missing option).

Comments

@pomek
Copy link
Member

pomek commented Nov 29, 2018

Is this a bug report or feature request? (choose one)

🐞 Bug report

πŸ’» Version of CKEditor

Latest master.

πŸ“‹ Steps to reproduce

  1. Insert a link in the editor.
  2. Hold CMD and click on the inserted link.

βœ… Expected result

Editing link toolbar should appear. The link shouldn't be opened in a new tab.

❎ Actual result

Clicked link is opened in the new tab.

πŸ“ƒ Other details that might be useful

I am not sure it is a bug or not. If not, the ticket is invalid.

@pomek pomek added the type:bug This issue reports a buggy (incorrect) behavior. label Nov 29, 2018
@pomek
Copy link
Member Author

pomek commented Nov 29, 2018

I forgot to mention about browsers… I can reproduce it on Firefox. On Chrome this behaviour does not occur. So it seems to be a bug.

@oleq
Copy link
Member

oleq commented Dec 7, 2018

I confirm that but...

...I also noticed this is a native feature of FF called "Open in Background Tab" https://support.mozilla.org/en-US/kb/mouse-shortcuts-perform-common-tasks, which can also be triggered by middle-clicking a link. I checked CKEditor 4 and it allows this behavior. I don't remember any bug reports concerning it in the past.

Anyway, neither Ctrl/Cmd+left click nor plain middle-click opens a new tab in Chrome. I checked that using plain contenteditable so it's not precisely about CKEditor 5.

TBH, I wouldn't be worried about that. Even if a user uses this shortcut accidently, the link opens in a background tab which does not disrupt the editing process. What's more, instead of disabling it in FF, I'd rather consider implementing the feature for Google Chrome because it feels like a nice UX to me.

cc @Reinmar

@oleq oleq added type:question This issue asks a question (how to...). status:discussion package:link and removed type:bug This issue reports a buggy (incorrect) behavior. labels Dec 7, 2018
@dkonopka
Copy link
Contributor

dkonopka commented Dec 7, 2018

What's more, instead of disabling it in FF, I'd rather consider implementing the feature for Google Chrome because it feels like a nice UX to me.

πŸ‘ πŸ‘ πŸ‘

@Reinmar Reinmar added this to the backlog milestone Mar 25, 2019
@mlewand mlewand added browser:firefox domain:ui/ux This issue reports a problem related to UI or UX. squad:core Issue to be handled by the Core team. labels Oct 1, 2021
@mlewand
Copy link
Contributor

mlewand commented Oct 1, 2021

I know this has been requested feature back in CKE4 (not the most popular one, but the kind that is asked every 2 months or so) and we made unofficial plugin on my account that people were happy with https://github.com/mlewand/ckeditor-plugin-openlink

@Reinmar
Copy link
Member

Reinmar commented Oct 1, 2021

Note: If we'll work on this, let's make sure we don't introduce any XSS issues.

@Reinmar
Copy link
Member

Reinmar commented Oct 13, 2021

  • Scope:
    • Cmd+click
    • Cmd+click already works in GH Writer, so it's rather feasible.
      • Question: What's the modifier on Windows?
  • Option+Enter should also open a link.
    • It works in Google Docs
    • What's the keystroke on Windows?
  • We don't need to worry if someone has some hard block for "open in new tabs".
  • Possible followup: Underline all links on "Cmd" press. GH Writer implements this feature.

@dawidurbanski dawidurbanski added type:task This issue reports a chore (non-production change) and other types of "todos". and removed status:discussion type:question This issue asks a question (how to...). labels Oct 14, 2021
@arkflpc arkflpc self-assigned this Oct 14, 2021
@Reinmar Reinmar modified the milestones: backlog, iteration 49 Oct 25, 2021
niegowski added a commit that referenced this issue Nov 2, 2021
Feature (link): Adds the possibility to open a link by Ctrl/Cmd+click or Alt+Enter. Closes #1381.
@AnnaTomanek AnnaTomanek added type:feature This issue reports a feature request (an idea for a new functionality or a missing option). and removed type:task This issue reports a chore (non-production change) and other types of "todos". labels Nov 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:link squad:core Issue to be handled by the Core team. type:feature This issue reports a feature request (an idea for a new functionality or a missing option).
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants