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

Add keyboard shortcut to show/hide sidebar #27994

Closed
A-t-s-u-s-h-i opened this issue Jan 24, 2023 · 15 comments · Fixed by brave/brave-core#17373
Closed

Add keyboard shortcut to show/hide sidebar #27994

A-t-s-u-s-h-i opened this issue Jan 24, 2023 · 15 comments · Fixed by brave/brave-core#17373

Comments

@A-t-s-u-s-h-i
Copy link

Description

I have a user who is considering to change his main browser from Vivaldi to Brave. He says Brave should have a shortcut key to show/hide sidebar. Now users need to move mouse cursor to the side of the browser window or click sidebar button to show/hide sidebar. But sometimes these operations could be frustrating users a bit. It would be very nice if Brave has the shortcut key to show/hide sidebar.

FYI, Vivaldi shows/hides sidebar by [option] + [command] + [p] on MacOS.

@rebron rebron added the needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. label Jan 27, 2023
@rebron
Copy link
Collaborator

rebron commented Jan 27, 2023

Proposing Ctrl+B on Windows and command+B on macOS to match Firefox open/close of sidebar.

@rebron rebron changed the title [FeatureRequest]New shortcut key to show/hide sidebar Add keyboard shortcut to show/hide sidebar Jan 27, 2023
@rebron
Copy link
Collaborator

rebron commented Feb 24, 2023

@simonhong simonhong self-assigned this Feb 25, 2023
simonhong added a commit to brave/brave-core that referenced this issue Feb 27, 2023
fix brave/brave-browser#27994

Ctrl+B(Cmd+B on macOS) toggles sidebar.
@brave-builds brave-builds added this to the 1.50.x - Nightly milestone Feb 27, 2023
@rebron rebron removed the needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. label Feb 28, 2023
@stephendonner
Copy link

stephendonner commented Mar 13, 2023

Verified PASSED using

Brave 1.50.90 Chromium: 111.0.5563.64 (Official Build) beta (x86_64)
Revision c710e93d5b63b7095afe8c2c17df34408078439d-refs/branch-heads/5563@{#995}
OS macOS Version 11.7.4 (Build 20G1120)

Steps:

  1. installed 1.50.90
  2. launched Brave
  3. clicked on the Show Sidebar button on the browser toolbar
  4. confirmed it opened
  5. pressed command + B
  6. confirmed the sidebar snapped closed
  7. pressed command + B again
  8. confirmed the sidebar opened
example example example
Screen Shot 2023-03-13 at 4 01 32 PM Screen Shot 2023-03-13 at 4 01 28 PM Screen Shot 2023-03-13 at 4 07 32 PM

Also tried the keyboard shortcut in various scenarious/states in 1.49.120, and it didn't do anything, as expected. All keyboard shortcuts with B also have additional modifiers:

example example
Screen Shot 2023-03-15 at 10 46 47 AM Screen Shot 2023-03-15 at 10 41 36 AM

@LaurenWags
Copy link
Member

@brave/qa-team please check keyboard shortcut on previous release (1.49.x) to confirm we're not overriding a previous keyboard shortcut as well as confirming the keyboard shortcut behaves as expected with 1.50.x.

@MadhaviSeelam
Copy link

MadhaviSeelam commented Mar 29, 2023

Verification PASSED using

Brave | 1.50.108 Chromium: 112.0.5615.39 (Official Build) (64-bit)
-- | --
Revision | a0e7b9718a92bcd1cf33b7c95316caff3fc20714-refs/branch-heads/5615@{#753}
OS | Windows 11 Version 22H2 (Build 22621.1413)

New Profile - Sidebar left

  1. Install 1.50.108
  2. launch Brave
  3. clicked on the Show Sidebar button on the browser toolbar
  4. confirmed it opened
  5. pressed Ctrl+ B
  6. confirmed the sidebar snapped closed
  7. pressed command + B again
  8. confirmed the sidebar opened
  9. clicked reading list icon to close the sidepanel and sidebar
step 4 step 6 step 8
image image image

Upgrade - Sidebar right

  1. Install 1.49.132
  2. launch Brave
  3. open brave://settings/appearance
  4. select Show on right option in `Show Sidebar button
  5. verified Ctrl+alt+B (bookmarks bar)
  6. close the browser
  7. relaunch Brave with 1.50.x
  8. clicked on the Show Sidebar button on the browser toolbar
  9. confirmed it opened
  10. pressed Ctrl+ B
  11. confirmed the sidebar snapped closed
  12. verified Ctrl+alt+B (bookmarks bar) again
step 4 step 8 step 10 ctrl+alt+B (bookmarks bar)
image image image image

@MadhaviSeelam MadhaviSeelam added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Mar 29, 2023
@btlechowski
Copy link

Verification passed on

Brave 1.50.93 Chromium: 111.0.5563.64 (Official Build) beta (64-bit)
Revision c710e93d5b63b7095afe8c2c17df34408078439d-refs/branch-heads/5563@{#995}
OS Ubuntu 18.04 LTS

New Profile - Sidebar left

  1. Install 1.50.x
  2. launch Brave
  3. clicked on the Show Sidebar button on the browser toolbar
  4. confirmed it opened
  5. pressed Ctrl+ B
  6. confirmed the sidebar snapped closed
  7. pressed ctrl + B again
  8. confirmed the sidebar opened
  9. clicked reading list icon to close the sidepanel and sidebar

image
image

Upgrade - Sidebar right

  1. Install 1.49.x
  2. launch Brave
  3. open brave://settings/appearance
  4. select Show on right option in `Show Sidebar button
  5. close the browser
  6. relaunch Brave with 1.50.x
  7. clicked on the Show Sidebar button on the browser toolbar
  8. confirmed it opened
  9. pressed Ctrl+ B
  10. confirmed the sidebar snapped closed

1.49.x
image

1.50.x
image
image

@bastian-purrer
Copy link

As a user that doesn't use the sidebar, this implementation is very flawed. I accidentally open the sidebar, with no obvious way to close it.
The sidebar is also opened if I explicitely de-activated it in the Brave settings. And there is no way to deactivate or change the shortcut. Major headache

@simonhong
Copy link
Member

As a user that doesn't use the sidebar, this implementation is very flawed. I accidentally open the sidebar, with no obvious way to close it. The sidebar is also opened if I explicitely de-activated it in the Brave settings. And there is no way to deactivate or change the shortcut. Major headache

@bastian-purrer I think you could remove unwanted shortcuts from brave://settings/system/shortcuts.
As it's available from 1.53.x(currently in beta), you could customize shortcuts from the next major update if you're using stable channel now. Hope that helps!
image

@q2apro
Copy link

q2apro commented Jun 21, 2023

The decision to user CTRL B is unfortunately bad.

Every time I am using an editor (HTML textarea) and mark a word, then hit Ctrl B, the word gets marked bold - ⚠️ AND the sidebar of Brave shows up. And the editor (textarea) is losing focus!

I cannot deactivate it under brave://settings/system/shortcuts

There are also no browser settings.

Please remove this shortcut again.

@simonhong
Copy link
Member

@q2apro Sorry for the inconvenience. We're planning to release major update next week.
With it. you could remove ctrl + b from brave://settings/system/shortcuts

@mihaime
Copy link

mihaime commented Jul 3, 2023

this one is super annoying especially in Confluence where you use ctrl+b / cmd+b to Bold Text.
Now instead the sidebar keeps popping up..

@q2apro
Copy link

q2apro commented Jul 3, 2023

The latest update did NOT solve the issue.

@elron
Copy link

elron commented Jul 5, 2023

Please re-open this issue.
The command+B (in mac devices) should be reserved to bold text in text editors, not open sidebar. (Personally I don't use the sidebar at all)

@mihaime
Copy link

mihaime commented Jul 5, 2023

same opinion here...adding a modified shortcut for "a single user" "who is considering to change his main browser from Vivaldi to Brave" does not really make it the norm for the rest of us that are really using Brave on a daily basis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment