Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Blank tabs bar appears after increasing the number of tabs per tab set #7806

Closed
luixxiul opened this issue Mar 21, 2017 · 7 comments
Closed

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Mar 21, 2017

Test plan

#9773 (comment)


Describe the issue you encountered:
the tabs bar becomes blank after increasing the number of tabs per tab set under a condition. I believe the same kind of issue had been fixed once and then regressed, recently or not.

  • Platform (Win7, 8, 10? macOS? Linux distro?):

  • Brave Version (revision SHA): 64b381a

  • Steps to reproduce:

    1. Open about:preferences#tabs
    2. Set the number of tabs per tab set to 6
    3. Open new tabs to create a new tab set, which has just 2 tabs
    4. Increase the number to 8
  • Actual result: the blank tabs bar which has zero tabs appears

screenshot 2017-03-21 23 07 46

  • Expected result: the 2nd tab set should be closed

  • Any related issues:

@cezaraugusto
Copy link
Contributor

can't repro per str, can you confirm?

@luixxiul luixxiul self-assigned this Jun 8, 2017
@bsclifton
Copy link
Member

@luixxiul is this still an issue? I'll close and assign to 0.17.x so we can retest

@bsclifton bsclifton added this to the 0.17.x (Beta Channel) milestone Jun 29, 2017
@srirambv
Copy link
Collaborator

This is still reproducible if the 8th tab opened is preferences and change the tabset count to 8 then you get the empty tabs bar
7806

This is been a known issue when you have tabsets and if you close the single tab in a new set it shows empty tabs bar. And the behavior is still same even if you have pinned tabs

@srirambv srirambv reopened this Jun 29, 2017
@srirambv srirambv modified the milestones: 0.19.x (Nightly Channel), 0.17.x (Beta Channel) Jun 29, 2017
@NejcZdovc NejcZdovc assigned NejcZdovc and unassigned luixxiul Jun 29, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 29, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 29, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
@NejcZdovc
Copy link
Contributor

@bsclifton I created a fix for this one, so if we want to pull this into 0.18, please change milestone for this issue and connected PR

NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 29, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
@luixxiul
Copy link
Contributor Author

should we pull the issue to 0.18 if the fix works?

@bsclifton
Copy link
Member

Pulling into 0.18.x sounds great (if the fix works and there are sufficient tests) 😄

@NejcZdovc
Copy link
Contributor

I am trying to add test, but context is messing with me 😄

NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 29, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
@NejcZdovc NejcZdovc modified the milestones: 0.18.x (Developer Channel), 0.19.x (Nightly Channel) Jun 30, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 30, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 30, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jul 7, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jul 9, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jul 9, 2017
Resolves brave#7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
NejcZdovc added a commit that referenced this issue Jul 11, 2017
Fixes tab pages after setting change
NejcZdovc added a commit that referenced this issue Jul 11, 2017
Resolves #7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
NejcZdovc added a commit that referenced this issue Jul 11, 2017
Resolves #7806

Auditors: @bsclifton @cezaraugusto

Test Plan:
- Open about:preferences#tabs
- Set the number of tabs per tab set to 6
- Open new tabs to create a new tab set, which has just 2 tabs
- Increase the number to 8
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.