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

Have a "hold to actually close" keyboard shortcut for pinned tabs #4963

Open
bsclifton opened this issue Jun 18, 2019 · 13 comments
Open

Have a "hold to actually close" keyboard shortcut for pinned tabs #4963

bsclifton opened this issue Jun 18, 2019 · 13 comments
Labels
feature/pinned-tabs priority/P5 Not scheduled. Don't anticipate work on this any time soon. suggestion

Comments

@bsclifton
Copy link
Member

bsclifton commented Jun 18, 2019

Description

Currently, pinned tabs can be closed with Ctrl+W / Cmd+W, and we've clarified this is the intended behavior.

A Chrome user recently suggested that we could have a "Hold X to close", similar to the Chrome/Chromium default behavior for Ctrl+Q/Cmd+Q for quit. It would look something like this:
image

Steps to Reproduce

  1. Fresh profile
  2. Visit a site
  3. Right click tab, pick Pin Tab
  4. Go into pinned tab
  5. Hit Ctrl+W or Cmd+W

Actual result:

Pinned tab closes ☹️

Expected result:

User should have to hold shortcut to close

Reproduces how often:

100%

Brave version (brave://version info)

Brave 0.68.56 Chromium: 75.0.3770.90 (Official Build) nightly(64-bit)
Revision a6dcaf7e3ec6f70a194cc25e8149475c6590e025-refs/branch-heads/3770@{#1003}
OS Mac OS X
@rebron
Copy link
Collaborator

rebron commented Jul 5, 2019

#4988 (comment)
Approach to solve inadvertently closing pinned tabs via ignoring CMD + W pref, might be a good way to go to instead of a warning.

@rebron
Copy link
Collaborator

rebron commented Jul 5, 2019

cc: @rossmoody

@rebron rebron added the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label Jul 5, 2019
@eljuno
Copy link

eljuno commented Dec 24, 2019

+1 from Community https://community.brave.com/t/possibly-quick-ux-improvement-for-pinned-tabs/99549?u=eljuno

@ibwoo
Copy link

ibwoo commented Mar 31, 2020

+1 from me - a great idea that would be very helpful!

@yuujay
Copy link

yuujay commented May 17, 2020

+1. It gets little annoying when a pinned tab gets closed accidentally. Would be a very helpful UX improvement.

@iQuery-001
Copy link

+1 from me yes please

@yigitkonur
Copy link

+1

@radiosilence
Copy link

radiosilence commented Jun 25, 2020

Much prefer the preference option. Often pinned tabs are closed when you spam/hold Cmd+W so having a message would probably be ineffective. What exactly is the point of pinning of they are so easy to close?

Someone suggested restoring the tab, but if you're like me and have a mix open on SoundCloud or whatever, it doesn't restore the playback.

@dvcrn
Copy link

dvcrn commented Jan 27, 2021

I like the idea of having a setting to disable ever closing pinned tabs, similar to how safari is doing it. You can still right-click and then hit "close", but pinned tabs should be pinned until they are explicitly unpinned.

Hitting CMD-W on a pinned tab, if that tab is the last tab, should just close the browser window and leave the tab as is.

@thebatvic
Copy link

thebatvic commented Jun 14, 2021

+1 from me.
It's already in WIP or it is still not take yet?

@gustavotrott
Copy link

I created an extension as a workaround to prevent pinned tabs from being closed accidentally:

https://github.com/gustavotrott/brave-lock-pinned-tabs

Feel free to use if it can help you guys to tackle this issue!

@rebron rebron moved this to P5 backlog in Front End May 28, 2024
@ricardoprato
Copy link

+1

@gofort
Copy link

gofort commented Nov 16, 2024

+1 for that feature
btw, extension posted above is a pretty great working workaround!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/pinned-tabs priority/P5 Not scheduled. Don't anticipate work on this any time soon. suggestion
Projects
Status: P5 backlog
Development

No branches or pull requests