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

Temporarily opened side panel is hard to close #34085

Closed
sangwoo108 opened this issue Nov 3, 2023 · 3 comments · Fixed by brave/brave-core#20915
Closed

Temporarily opened side panel is hard to close #34085

sangwoo108 opened this issue Nov 3, 2023 · 3 comments · Fixed by brave/brave-core#20915
Assignees
Labels
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. QA Pass-Win64 QA/Yes release-notes/include

Comments

@sangwoo108
Copy link

Reported https://github.com/brave/reviews/issues/1421#issuecomment-1791478022

Ok thanks I'll test it out. In the meantime I think I found another UX bug:

1. In settings, change "Sidebar" to "Never" and disable the sidebar button. 
2. Go to a youtube video and click add to playlist then open in playlist.
3. The sidebar is now open with no way to close it.

You can solve this by going to settings, enabling the sidebar button, clicking it, then disabling it again, but the UX is not very good. I imagine this would annoy the users who have disabled sidebar entirely.

EDIT: looks like you can also close it by clicking the playlist icon in the sidebar but this wasn't obvious - maybe need an X button on the sidebar to close it explicitly
If user disables side panel location bar button and playlist icon from sidebar, it's difficult to close.
How about bring playlist item back when user add item to playst from omnibox?
@rebron rebron added feature/sidebar Relating to Brave's Sidebar feature priority/P3 The next thing for us to work on. It'll ride the trains. release-notes/include QA/Yes labels Nov 3, 2023
@simonhong
Copy link
Member

@rebron @sangwoo108 How about doing this generally not just for playlist?
Other side panel could be opened w/o sidebar item interaction.
Ex, leo panel could be opened from omnibox.

@sangwoo108
Copy link
Author

sangwoo108 commented Nov 9, 2023

Yes, that was exactly what I was thinking of.
https://github.com/brave/reviews/issues/1421#issuecomment-1791884877

It should be general behavior 👍

@MadhaviSeelam
Copy link

MadhaviSeelam commented Nov 13, 2023

Verification PASSED using

Brave | 1.62.45 Chromium: 119.0.6045.134 (Official Build) nightly (64-bit)
-- | --
Revision | 7853f6ed48cd6ea40b7baf3654b72b57193d32c5
OS | Windows 11 Version 22H2 (Build 22621.2428)
  1. Installed 1.62.45
  2. launched Brave
  3. verified #brave-ai-chat is enabled as a default in brave://flags
  4. verified Leo icon is shown on the sidebar as a default`
  5. right clicked on Leo button and removed Leo item from sidebar
  6. confirmed Leo button is removed
  7. typed any keyword (crypto) at omnibox
  8. selected Leo omnibox result
  9. confirmed Leo sidepanel opened
  10. confirmed Leo button is added to the sidebar and shown as activated/highlighted
  11. clicked Accept & begin
  12. confirmed crypto keyword shown in the sidepanel
  13. closed the sidepanel via sidebar toolbar button
  14. confirmed sidepanel closed and Leo button is still shown
step 3 step 4 step 5 step 6 step 7 step 9 step 12 step 14
image image image image image image image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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. QA Pass-Win64 QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants