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

Work required before we can release Sidebar #14980

Closed
18 of 28 tasks
bsclifton opened this issue Mar 26, 2021 · 13 comments
Closed
18 of 28 tasks

Work required before we can release Sidebar #14980

bsclifton opened this issue Mar 26, 2021 · 13 comments
Assignees
Labels
closed/no-milestone feature/sidebar Relating to Brave's Sidebar feature OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains.

Comments

@bsclifton
Copy link
Member

bsclifton commented Mar 26, 2021

Description

We have a Sidebar that is available in Nightly which can be enabled via brave://flags/#sidebar

Some specific issues have been captured and labeled with feature/sidebar

This issue is intended to track work required before we can release an MVP (minimal viable product) of sidebar

P1

P2

P3

  • Resizable sidebar panel
  • Sidebar item missing selected state when open in the current tab (Need more discussion for this item)
  • Clicks and hold on the sidebar item should show options (Show sidebar item's menu with long press #16017)
  • Sidebar options menu should use Brave menu component (applies to all sidebar icon menus and show sidebar menu)
  • Show sidebar on top of webview rather than resizing the webview (Show sidebar on top of webview rather than resizing the webview  #16059)
  • Add to sidebar by dragging the site information (lock) icon in the URL bar into the sidebar
  • Add to sidebar by dragging the tab into the sidebar
  • Drag a sidebar icon off to remove it
  • Need focus states
  • Incorrect typography treatments on “Add to Sidebar” menu

BUG

@bsclifton bsclifton added OS/Desktop feature/sidebar Relating to Brave's Sidebar feature labels Mar 26, 2021
@bsclifton
Copy link
Member Author

bsclifton commented Mar 26, 2021

Some other concerns which came up:

  • No URL is shown for content loaded in sidebar. This could be a problem as there's no way to ensure which site you're on
  • Trying to use functionality which requires microphone/camera (ex: Brave Together) will ask for permission... but this ends up being hidden because there is no visible omnibox for the sidebar (where you'd typically see the permission requests show up). Maybe we can have a "Permissions UI" area of sidebar for this?

@srirambv
Copy link
Contributor

#14402 should be part of MVP as well (just a suggestion)

@NumDeP
Copy link

NumDeP commented Jun 6, 2021

I hope you don't mind me adding just a few more additional features/issues/tests

  • Purposely cause a crash and see if it asks to restore and does it actually restore on the sidebar and not the normal tab place.

  • Does it allow to reopen closed tabs, including in private tabs as well (it's going on 2 years soon or is over 2 years since a fix for this has come to horizontal tabs, still waiting for upstream fix)

  • When you've set 'Continue where you left off' and close the window/browser, do the tabs open on sidebar

  • Indicate the number of tabs open near "OPEN TABS". Perhaps you could indicate it like "67 OPEN TABS" (This would come in handy if it was implemented before crash test; I've occasionally had restore not work effectively for me)

  • Right-click - Bookmark tab to... / Bookmark all tabs to...

  • Some tabs shouldn't be easily removed/dragged as suggested above without confirmation, case and point - webtorrent. Not that I use it regularly but the UX for this feature isn't great if you'll pardon my bluntness and closing such a tab with a file on download or downloaded can be frustrating. When this is toggled/On and other similar proprietary brave solutions, perhaps they should always remain pinned to the top.

  • If horizontal tabs is going to be redundant by the use of the sidebar tabs, is there any point to have the empty bar left over from the horizontal tabs. I mean, with the exception of the minimize, maximize and close button which you can push down, is there any point of an empty bar, it's a waste of space right? Maybe be linked to - (Feature Request: Hide all the tabs #16384)

@AshrayaCoder
Copy link

When Might This Drop?
Any Estimations?

@JeswinSunsi
Copy link

Any updates on this? 👀

@plandem
Copy link

plandem commented Feb 8, 2022

is it possible to prevent reloading when I switch apps on siderbar? right now it fresh reload everytime I switch app

@JeswinSunsi
Copy link

Hey is the sidebar still limited to those 4 tabs?

@niutech
Copy link

niutech commented Jun 29, 2022

Could you please open the web type items (websites added to the sidebar) in an overlay instead of the current tab, like it is in Opera?

Opera sidebar

We already have a bookmarks bar for opening websites in the current tab.

@IMrCreepyI
Copy link

IMrCreepyI commented Jul 2, 2022

just wondering how far the progress is cause im really looking forward to this.
if its still being worked on

@JeswinSunsi
Copy link

This isn't a sidebar though. These are bookmark icons on the side. Something like the one in Opera would be very useful to many users. Only reason I haven't switched to OperaGX yet is privacy.

@simonhong
Copy link
Member

just wondering how far the progress is cause im really looking forward to this. if its still being worked on

@IMrCreepyI Sidebar feature is already available. It's not shown by default on Stable release.
You can enable it via app menu.
image

@IMrCreepyI
Copy link

just wondering how far the progress is cause im really looking forward to this. if its still being worked on

@IMrCreepyI Sidebar feature is already available. It's not shown by default on Stable release. You can enable it via app menu. image

i have that enabled but i mean like what they have in operaGX and in a message above they show a picture of it

@bsclifton
Copy link
Member Author

Closing issue as this is now live 🎉 We can open individual issues for specific items we'd like (that aren't already implemented)
cc: @rebron in case we needed to file items

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/no-milestone feature/sidebar Relating to Brave's Sidebar feature OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains.
Projects
None yet
Development

No branches or pull requests