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

Pinned tabs lose order after re-launch #8543

Closed
danielrob opened this issue Apr 28, 2017 · 33 comments
Closed

Pinned tabs lose order after re-launch #8543

danielrob opened this issue Apr 28, 2017 · 33 comments

Comments

@danielrob
Copy link

danielrob commented Apr 28, 2017

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Updated Brave today and after relaunch my pinned browser tabs were no longer in the correct order. Thought it was an update bug, so put them back in order, but after quitting and re-launching they returned to the same incorrect order.

  • Platform / Revision

Brave: 0.15.0 
rev: 71fb73001617949528cc6473e288a49e76fcd4d5 
Muon: 2.58.7 
libchromiumcontent: 58.0.3029.81 
V8: 5.8.283.32 
Node.js: 7.9.0 
Update Channel: dev 
os.platform: darwin 
os.release: 15.4.0 
os.arch: x64
  • Steps to reproduce:

    1. Pin some tabs,
    2. Change pin order,
    3. quit, re-open
  • Actual result:
    Pinned in different order

  • Expected result:
    Pinned in same order as when closed.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Hard to say, I don't want to wipe my profile / haven't set up a dev environment yet.

  • Is this an issue in the currently released version?
    yes

  • Can this issue be consistently reproduced?
    yes, but as per above.

  • Screenshot if needed:
    https://cl.ly/1K2w1G03162O

@lukemulks
Copy link
Collaborator

We've had an additional report of this in the Brave Community:
https://community.brave.com/t/oh-my-god-stop-with-the-tabs-to-new-windows/2473/6?u=luke.mulks

Feedback

Just having a bit of an issue with the tabs since the latest update.

The pinned tabs are in different order, when repositioned and open a new window, they still show in the wrong order.

Also, when open a new window, normally the new tab in the window should be the active one, instead it's the last one of the pined ones. I have checked the preferences and the new tab is expected to be the active one.

Browser Profile

Brave	0.15.0
rev	71fb730
Muon	2.58.7
libchromiumcontent	58.0.3029.81
V8	5.8.283.32
Node.js	7.9.0
Update Channel	dev
os.platform	darwin
os.release	16.6.0
os.arch	x64

@lukemulks
Copy link
Collaborator

Update: We've released an update (v0.15.1) this evening that addresses some tab issues.

I've checked with the two Brave Community contributors that experienced/observed the issue, and requested that they check to see if v0.15.1 resolves for them.

Community URL:
https://community.brave.com/t/oh-my-god-stop-with-the-tabs-to-new-windows/2473/8

I'll report back here with what the contributors in the Community respond back with after they test.

@Jacalz
Copy link
Contributor

Jacalz commented Apr 30, 2017

+1 from community: https://community.brave.com/t/pinned-tab-shuffle-and-lost-focus-changing-tabs-in-v0-15-0/2522/2
Still not fixed with 0.15.1 sadly 😕

@rojour
Copy link

rojour commented Apr 30, 2017

Not resolved for me :(

Name Version
Brave 0.15.1
rev ea5024d
Muon 2.58.8
libchromiumcontent 58.0.3029.81
V8 5.8.283.32
Node.js 7.9.0
Update Channel dev
os.platform darwin
os.release 16.6.0
os.arch x64

@danielrob
Copy link
Author

Still broken here too.

Brave: 0.15.1 
rev: ea5024d1f3e47d34de79b70e3cd74f1408c985ce 
Muon: 2.58.8 
libchromiumcontent: 58.0.3029.81 
V8: 5.8.283.32 
Node.js: 7.9.0 
Update Channel: dev 
os.platform: darwin 
os.release: 15.4.0 
os.arch: x64

@lukemulks
Copy link
Collaborator

The other Brave Community contributor confirmed re-ordering still remains an issue in 0.15.1

Community URL:
https://community.brave.com/t/oh-my-god-stop-with-the-tabs-to-new-windows/2473/15?u=luke.mulks

Still not working for me

Brave	0.15.1
rev	ea5024d
Muon	2.58.8
libchromiumcontent	58.0.3029.81
V8	5.8.283.32
Node.js	7.9.0
Update Channel	dev
os.platform	win32
os.release	10.0.15063
os.arch	x64

Note: 0.15.1 wasn't specifically pushing a fix for this issue, but did include some changes to tab behavior, so given that some contributors were reporting this issue on the same thread as another tab issue in Community I had requested a quick check to see if this issue persisted.

@bsclifton
Copy link
Member

Hi all-

I just merged a great fix by @romamatusevich; this fix will be in our next release, 0.15.2 😄

@danielrob
Copy link
Author

danielrob commented May 6, 2017

I can't reopen this issue it seems, but this is re-broken. Same repro steps.

Brave: 0.15.2 
rev: af7ef42ad622f899312a38bdd0e2138154393627 
Muon: 2.58.9 
libchromiumcontent: 58.0.3029.96 
V8: 5.8.283.37 
Node.js: 7.9.0 
Update Channel: dev 
os.platform: darwin 
os.release: 15.4.0 
os.arch: x64

@luixxiul luixxiul reopened this May 6, 2017
@luixxiul
Copy link
Contributor

luixxiul commented May 6, 2017

The fix should be ready on the next release, which is 0.15.3 (0.15.2 is a hotfix release).

@rojour
Copy link

rojour commented May 27, 2017

I waited forever to get 0.15.3 and the issue is not fixed yet. Tabs are now in a different order that in 0.15.1 (change by themselves) and if you reorder them, when the window is closed, if you open a new window, the order is back to what it was....

I also need to say that having the tabs pined, makes opening a new page sooooooo slooooooow. It will not allow anything to be done until pages are loaded... I have 7 of them and it is a pain in the wasooo...

I love the browser, but since the 0.15.1 update, it's not been that fun.

@danielrob
Copy link
Author

danielrob commented May 28, 2017

@rojour is correct.

I thought this was perhaps a migration problem for existing users.

I moved /Users/daniel/Library/Application Support/brave to brave-backup and started over. With two tabs, it worked, so I reinstated my standard dock of ~20 tabs. (I love using Brave pinned tabs as my 'web-dock'). On reboot the tab order was neither the order of pinning, nor the post-pinning selected order. I've retested after a fresh download, and using 3 tabs too. The same repro steps apply, and this is not fixed.

Brave: 0.15.310 
rev: 6b5e4e2d300959848e0e848817aa40385447f2eb 
Muon: 3.0.201 
libchromiumcontent: 58.0.3029.110 
V8: 5.8.283.38 
Node.js: 7.9.0 
Update Channel: dev 
os.platform: darwin 
os.release: 15.6.0 
os.arch: x64

@romamatusevich
Copy link
Contributor

romamatusevich commented May 30, 2017

I can confirm that this issue is actual for the latest master branch. Also I did testing for the commit 31e8cdd which includes initial fix and can't reproduce this issue. Looks like this issue was reintroduced later.

@srirambv
Copy link
Collaborator

Order of pinned tabs changes when a new window is launched. No console errors seen as mentioned by @LaurenWags . This is on the latest 0.18.23 upgrade.

pinnedtabs

@NejcZdovc
Copy link
Contributor

@LaurenWags your error should be fixed with #10591

@hugobuddel
Copy link
Contributor

The order of tabs, whether pinned or not, seems to change arbitrarily. That is, it is very hard to order tabs, especially the pinned ones, because the tab jumps to a seemingly random location as soon as I drop the tab in the preferred location. The pinned tabs are in a different order for each window I open. Furthermore, normal tabs also reorder as soon as I switch tabs, assuming I've manually rearranged them (not sure whether they switch back to their original order).

Using 0.18.36 on Windows 10. Having the pinned tabs (a feature I only discovered recently) makes the reordering much less annoying, because the 'normal' tabs will never interfere with pinned tabs. Actually, pinned tabs might be the feature I love the most about brave.

@LaurenWags LaurenWags removed their assignment Sep 22, 2017
@petemill petemill self-assigned this Oct 24, 2017
@bbondy bbondy modified the milestones: 0.21.x (Developer Channel), 0.20.x (Beta Channel), Backlog Oct 25, 2017
@alexwykoff alexwykoff added priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). priority/P4 Minor loss of function. Workaround usually present. and removed priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). labels Oct 31, 2017
@bbondy bbondy modified the milestones: Triage Backlog, Prioritized Backlog Nov 2, 2017
@srirambv
Copy link
Collaborator

srirambv commented Dec 5, 2017

@bsclifton
Copy link
Member

Fixed with #13271

@bsclifton bsclifton modified the milestones: Backlog (Prioritized), 0.21.x (Beta Channel - CR 65) Feb 23, 2018
@bsclifton bsclifton modified the milestones: 0.21.x w/ Chromium 65 (Beta Channel), 0.22.x (Developer Channel) Mar 9, 2018
@srirambv
Copy link
Collaborator

srirambv commented Mar 22, 2018

Verified on Windows x64

  • 0.22.6 e6ff4ea
  • libchromiumcontent: 65.0.3325.162
  • muon: 5.1.0

Verified on macOS 10.12.6 x64 using the following build:

  • 0.22.6 e6ff4ea
  • libchromiumcontent: 65.0.3325.162
  • muon: 5.1.0

Verified on Ubuntu 10.10 x64

  • 0.22.7 8bb7e77
  • libchromiumcontent: 65.0.3325.181
  • muon: 5.1.1

@tadeasbucha
Copy link

I have downloaded latest version of Brave (as a possible new user) 0.22.13 on Win10 64bit and I can't order my tabs, neither the pinned ones neither normal ones. Also if I pin it doesn't put it to the most right side, but assigns the tab some random position.

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