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

Use vertical Tabs + Tab Scrolling flag = Brave doesn't open #28877

Closed
ghost opened this issue Mar 3, 2023 · 6 comments · Fixed by brave/brave-core#17461
Closed

Use vertical Tabs + Tab Scrolling flag = Brave doesn't open #28877

ghost opened this issue Mar 3, 2023 · 6 comments · Fixed by brave/brave-core#17461
Assignees
Labels

Comments

@ghost
Copy link

ghost commented Mar 3, 2023

Description

@sangwoo108 Brave doesn't open/breaks when when Tab Scrolling brave://flags/#scrollable-tabstrip flag is enabled and Use vertical tabs is on.
This doesn't happen when using normal horizontal tabs, so the only workaround is to switch to horizontal tabs before closing the browser, if not, then the easiest solution would be to edit the Local State file to disable the flag or edit Preferences file to disable Use vertical tabs.

Steps to Reproduce

  1. Enable brave://flags/#scrollable-tabstrip
  2. Enable Use vertical tabs
  3. Restart
  4. Brave doesn't open.

Brave version (brave://version info)

1.51.2 Chromium: 111.0.5563.50 (Official Build) nightly (64-bit)

@ghost ghost added the OS/Desktop label Mar 3, 2023
@sangwoo108 sangwoo108 self-assigned this Mar 3, 2023
@sangwoo108 sangwoo108 added the priority/P2 A bad problem. We might uplift this to the next planned release. label Mar 3, 2023
@sangwoo108
Copy link

Thanks for reporting. I'll take a look ASAP.

@Ayern12
Copy link

Ayern12 commented Mar 3, 2023 via email

@sangwoo108
Copy link

Sorry for the inconvinience. I made a patch so that you can open browser. But scrollable tab strip flag is not fully supported yet under vertical tab strip 😥 Will take a look soonish.

@sangwoo108
Copy link

The follow-up issue is here: #28906

@ghost
Copy link
Author

ghost commented Mar 6, 2023

Thanks for the quick fix and your work @sangwoo108.

PS. I love the new Shared pinned tab feature. Great work there too and Vertical tabs. 😁👌

@stephendonner
Copy link

Verified PASSED using

Brave 1.51.85 Chromium: 112.0.5615.49 (Official Build) beta (x86_64)
Revision bd2a7bcb881c11e8cfe3078709382934e3916914-refs/branch-heads/5615@{#936}
OS macOS Version 11.7.6 (Build 20G1231)

Steps:

  1. installed 1.51.85
  2. launched Brave
  3. opened brave://flags
  4. confirmed default for brave://flags/#brave-vertical-tabs
  5. confirmed default for brave://flags/#scrollable-tabstrip
  6. enabled both of the above
  7. clicked Relaunch
  8. loaded msnbc.com
  9. context-clicked and chose Use vertical tabs
  10. confirmed no crash
  11. loaded a few other sites and continued to use vertical tabs

Confirmed that vertical-tab scrolling, when enabled, no longer breaks Brave (it's known that it needs work, still)

example example example example example example
Screen Shot 2023-04-11 at 3 35 46 PM Screen Shot 2023-04-11 at 3 36 02 PM Screen Shot 2023-04-11 at 3 36 10 PM Screen Shot 2023-04-11 at 3 37 47 PM Screen Shot 2023-04-11 at 3 37 56 PM Screen Shot 2023-04-11 at 3 40 25 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants