brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.52k stars 2.27k forks source link

[Desktop] Users should be able to customize keyboard shortcuts #354

Closed bsclifton closed 1 year ago

bsclifton commented 6 years ago

Carried over from browser-laptop-

Users should be able to change/remap/clear keyboard shortcuts in the browser

As a work-around, we can consider including or linking to an extension such as Shortkeys that allows users to do this

vinicentus commented 6 years ago

+1

slavpetroff commented 5 years ago

+1

Jay1 commented 5 years ago

Hotkeys to navigate to recent tab is very important in my workflow, I've been using Brave exclusively for about 2 months now it's honestly the only small irritant with Brave browser so far.

AndresPrez commented 5 years ago

The fact that the Brave Browser gains control over the play/pause button (MacBook) is very annoying. I enjoy Brave but unfortunately this problem is a no go on the browser for now.

bsclifton commented 5 years ago

@AndresPrez there is a separate issue tracking media keys being taken over (which happens because of Chromium). Can you try the work-around suggested here? https://github.com/brave/brave-browser/issues/3807#issuecomment-475332062

AndresPrez commented 5 years ago

Awesome!, there's a bunch of configurations here that I didn't know.

I'm sure this simple answer: "brave://flags". Might solve a lot of issues out there.

SeevenDev commented 5 years ago

@bsclifton Hi! Any plan on implementing this?

The Shortkeys extension doesn't allow to clear default shortcuts (in my case, I'd like to use Ctrl+Shift+Del with Streamkeys, but this shortcut is already taken for clearing browsing data, which is really annoying).

orish1 commented 5 years ago

I've had the same issue with Chrome (and now Brave, thinking at least another developer would have addressed it).

I'm constantly deleting things by mistake because the default shortcut for adding a bookmark is Ctrl+D. What's worrying is that I sometimes don't even know what I've deleted because another window might be active. So when the bookmark dialog doesn't show up, I press Ctrl+D a few more times.

This is not the only problem I have with common shortcut keys, but this is the most serious one and I've lost valuable work and contacts and emails, etc without realizing until it's way too late. (Yes, I've even backed up data after I've deleted something important and only realized when I tried to recover the data from the backup!!!)

It's a habit I can't get rid of because I use the keyboard much more than I use my mouse - my work is very text-intensive. So I need an alternative shortcut for adding bookmarks for my research.

Any ideas in the meantime (shortkeys doesn't support adding bookmarks as a shortcut)?

Thanks

pstephenwille commented 5 years ago

This is a deal killer for me. Vivaldi is the only browser that supports customized key bindings.

ghost commented 4 years ago

I want to add shortcut for my readerView extension or the distill page function... Anyone know if there is some magic javascript I can drop into shortkeys to do this?

loneconspirator commented 4 years ago

I would like to assign keyboard shortcuts to bookmarks without using an untrusted extension

tommyjcarpenter commented 4 years ago

I agree to this issue. On mac, typing command w is very awkward on my Razer keyboard to close the current tab. Cant be done with one hand (the command key and the w are basically vertically on top of each other). Would love to change to shift w

vindberg commented 4 years ago

Please add this to the backlog for the next release! Its important for further adoption of the browser ASAP!!!

zwimer commented 4 years ago

I agree. I constantly hit ctrl-D to close a window (I am used to doing this for terminals). I would like the ability to change the bookmark key-binding to a different shortcut.

vinicentus commented 4 years ago

Would it be possible to write this as an open source chromium extension that would then be included by default? That way it might be easier to maintain. I don't know how efficient that would be, just throwing the idea to you :)

Exr0n commented 4 years ago

Or, is there a plan on implementing this? That might help somebody looking to contribute take a crack at it.. :)

gerardb7 commented 3 years ago

Having a mechanism to modify and clear default shortcuts would prevent a lot of nagging issues such as unwanted zooming when pressing ctrl. Please implement this!

auriusbendikas commented 3 years ago

This is the only reason now I am not using Brave browser. Not being able to customize shortcuts is a deal braker.

Would also be nice if I could edit shortcuts in a file, that would help me to scrip my browser installations with my customisations pre-applied.

brandon-toner commented 3 years ago

+1 — I'm getting interference with keystrokes I use with other web-apps (Roam Research).

Will be changing browsers because of this. May return if fixed.

developersam1995 commented 3 years ago

+1, Yes please add the ability to customize shortcuts.

impshum commented 3 years ago

+1

I'd like to map alternative shortcut keys easily too.

Example: Shift + l = refresh etc etc

That would be great!

Gutie commented 3 years ago

+1

Must-have for great ux.

trevorackerman commented 3 years ago

+1

Jaco7Fourie commented 3 years ago

+1

Al4ndil commented 3 years ago

Yeah ! It will be great ! Shortcut keys for Brave and extensions !

frandoodle commented 3 years ago

+1 Google docs shortcut for adding a comment is Ctrl + Shift + M which is overridden by Brave's account/wallet feature's (?) hotkey.

edmundlaugasson commented 3 years ago

+1 also in Notion these are interfering with Brave shortcut keys, e.g. CTRL+SHIFT+9 will move tab to left but should create a page in Notion. Here are Notion shortcut keys. Even Shortkeys didn't help me to remove it :(

sokkit commented 3 years ago

+1 also running in to issues with Brave and Notion. Would be really nice to be able to remove/change keyboard shortcuts.

NotRexButCaesar commented 3 years ago

The alt-shortcuts can also interfere when entering Unicode characters.

ECon87 commented 3 years ago

Is this happening? It would be great if we could customize shortcut keys

Ultrahalf commented 3 years ago

It would be cool if you could control multi media with keyboard. Idk what this thing is called image

TheYellowArchitect commented 3 years ago

While many uses are posted above, it would help Vim users as well (vim-like bindings, j-k down-up)

Optimally, a configuration text file would be appreciated, so the configuration can be moved to other computers, but honestly as long keyboard shortcut functionality is added, I wouldn't have anything else to really ask.

yamabiiko commented 2 years ago

+1 This feature would be a deal breaker, please give us an update

SINE commented 2 years ago

This is a vital functionality missing. The browser is using the same hotkeys for closing the browser as other programs use for unfocusing. When using with Parsec, this force closes the browser which is frustrating and a loss of time and productivity. PLEASE add customizable hotkeys or the ability to disable them at least.

Stijn-Hoeke commented 2 years ago

+1, would also want this for mouse shortcuts though.

LeuschkeTressa commented 2 years ago

Seems Brave doesn't respect macOS system shortcuts, at least not when they override a Brave shortcut. The custom keyboard shortcut is indicated in the corresponding Brave menu item, but has no effect.

At least a fix for that would be great.

tetsugakusha256 commented 1 year ago

+1, I would love to be able to freely remap the shortcuts.

romanr commented 1 year ago

It's like the worst possible implementation of keyboard shortcuts - not customisable and can not be overriden even by OS.

ghost commented 1 year ago

@fallaciousreasoning Your work is amazing, the only problem I have noticed is with Back and Forward navigation. They are set to left and right keys by default, which can cause issues while editing text and all that, but the problem is how they can't be completely removed, because when you remove them, they come back on restart, and if you hit reset button, then they glitch and Brave doesn't even allow to be removed them anymore, because they come back immediately. But it is working good besides that.

fallaciousreasoning commented 1 year ago

Oooh interesting, that sounds like a few bugs :laughing:. Could you provide a few details (this isn't working at all on macOS yet). What platform are you on? OSX, Linux, or Windows? What version of Brave are you using? (if you go to brave://version it should tell you).

Glad to hear you're already using it! It should help us iron out a bunch of the bugs and get things ready for launch.

edit: I've filed this issue https://github.com/brave/brave-browser/issues/29374, which should be fixed by https://github.com/brave/brave-core/pull/17810

ghost commented 1 year ago

@fallaciousreasoning

Thanks! yeah I have been using since the 4 minutes after the build (Windows) was available to download (I usually manually install builds minutes after they are available, so I usually have the latest).

I was just waiting for the Shortcuts Generation Followup (17632) that was released today to report this, I didn't know if this problem would get fixed by that. Only 3 commands are affected by this anyway, Back and Forward and Focus Menu Bar commands, so should be easy to test when the fix is available, and it was only annoying when editing URLs on the omnibox mostly if I forgot to remove them on every start.

Another thing I see is duplicates of shortcuts, which is like whatever, because removing both shortcuts works fine, plus it is only some commands which I don't really edit: Select Tab [0 to 7], Select Last Tab, Zoom Normal, Zoom Minus, Focus Menu Bar (as well).

But everything I have tested works okay with the commands that are meant to work in Brave, should be amazing when the design for the Shortcuts UI (looks really good!) gets implemented as well!

fallaciousreasoning commented 1 year ago

Wow, that's pretty fast!

Yeah, the duplicates are something I know about - they're actually not duplicated shortcuts but a different keys which have the same friendly names. For example Digit0 vs NumPad0, or Plus vs NumPadPlus. It definitley looks weird :laughing:

The UI update is coming along pretty nicely, but I'm waiting on some new components to land in our UI library, Leo, before I get it reviewed :smile:

ghost commented 1 year ago

@fallaciousreasoning whatever build has nice commits I set a timer because I already know when it will be available for Windows. But I have had the flag enabled when it was only viewing the commands was released. Nice news for the UI part, and I will be probably the first user to use it for sure!

Oh yes, I thought I said 'probably visual' in my comment, but then, I also forgot Preferences file exists and I could just have checked it to see exactly what you say 🤦‍♂️

            "34019": [
                "Control+Digit2",
                "Control+Numpad2"

Seems like 39003 = Focus Menu Bar, is crazy in the Accelerators in preferences file with 53 "AltLeft" image

So I guess it was getting crazy there! but I am glad it was or seems like it was a quick fix. So now, it's just about waiting to enjoy more this amazing feature 👍

Thanks again!

ghost commented 1 year ago

@fallaciousreasoning Back and Forward navigation working great now, thanks for the quick fix! 🎉

Focus Menu Bar, is the only one that I still see with issues, not as important because not many would change that shortcut but here is:

brave_2023-03-29_10-32-09

Another thing I noticed about it is that the Preferences file shows it like this: "39003":["F10","Alt","Alt","AltGr"], but if you remove the last Alt on UI = AltGr, the AltGr still works to focus in the menu bar, if you delete the first Alt, then that stops both Alt and AltGr from focusing in the menu bar, so it seems it is not getting the same treatment as the Back and Forward navigation, which they now even display like AltGr + ArrowLeft - in the UI to indicate the difference.

Edit: Also, trying to assign AltGr doesn't work, because when you press the AltGr key it shows as "Control" in the 'recorder', but if you press Control and then AltGr, it will say Control + AltGraph but when you accept, only adds Control to it. Probably a Windows thing, and it is easy just to go to Preferences and add AltGr a shortcut if wanted, but that's another thing I noticed.

fallaciousreasoning commented 1 year ago

Yeah, there's definitely a bit of funkiness :laughing: The Focus Menu Bar one is interesting because the shortcut is Alt with no other keys, and it actually doesn't work properly, at all (not in the Brave versions without the commands feature, not with the flag off, not in Chromium) which seems odd (especially because F10 works).

I think the duplicates probably occur because Chromium adds a shortcut for VKEY_MENU, VKEY_LMENU and VKEY_RMENU which (I think?) are all Alt/AltGr. I might have a bit of a dig into it.

The recorder definitely has a few problems too - one I've been noticing is that Meta doesn't show up until you press another key (but that might be a Linux thing). It's weird, but some input events don't fire (but then, when you press another key it knows what modifiers are down). I've been playing around with this thing to see if I can make sense of what's going on.

Thanks heaps for all the testing and feedback!

rebron commented 1 year ago

Closing out in favor of https://github.com/brave/brave-browser/issues/30840. Please add new comments there or file new issues not covered in 30840.

A few notes: We're targeting to release this feature within the 1.56.x time frame, still need to turn the flag on by default. We do have shortcuts on by default in Beta 1.57.x and Nightly 1.58.x.