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

Wrong next tab selected after stashing active tab #369

Closed
tokumeiii opened this issue Jun 9, 2023 · 10 comments
Closed

Wrong next tab selected after stashing active tab #369

tokumeiii opened this issue Jun 9, 2023 · 10 comments
Labels
i-bug Something might be broken in Tab Stash
Milestone

Comments

@tokumeiii
Copy link

tokumeiii commented Jun 9, 2023

System Details

  • OS Version: Windows 10 22H2
  • Browser Version: Firefox Nightly 116.0a1 (2023-06-08)
  • Tab Stash Version: 2.12.1

Installed Browser Extensions:
I tried disabling all extensions except for Tab Stash and the problem persisted.

Describe the Bug

When I stash the active tab by any means I know of to do it (address bar button, toolbar icon, tab context menu, the main Tab Stash sidebar), the tab seven tabs to the right of the stashed tab is selected as the new active tab.

Steps to Reproduce

  1. Stash the active tab.

Expected Behavior

The immediate next tab should be selected as the new active tab as happens when a tab is closed normally.

Impact

When selectively stashing multiple tabs at a time this throws you around all over the place and makes it hard to keep track of what tabs you've looked at already.

Additional Details

I happen to have a Linux VM handy with Firefox 114, and I was unable to reproduce the issue there.


Readers: If you are also experiencing this bug, please vote for it by giving the ORIGINAL POST a thumbs-up using the 😃 button. You are welcome to leave comments and discuss the issue if you have more details to add, but "Me too!" comments are not counted by the voting system.
@josh-berry
Copy link
Owner

Hi, sorry for the delay in getting back to you. Given that you're not able to reproduce the issue in a production release of Firefox, and I'm similarly not able to reproduce the issue on my side, my strong suspicion is this is not a problem with Tab Stash at all but instead a bug in Nightly, which is known to be unstable and have lots of weird issues.

If you can reproduce the issue on a production release of Firefox, please do let me know though!

@josh-berry josh-berry closed this as not planned Won't fix, can't repro, duplicate, stale Jun 18, 2023
@josh-berry josh-berry added the r-invalid This doesn't seem right label Jun 18, 2023
@davidcreel
Copy link

davidcreel commented Oct 22, 2023

@josh-berry Thank you for making this extension! I have observed this behavior fairly regularly with long running windows with lots of tabs. The behavior was present in Tab Stash 2.x and is still present in 3.x.

I have two windows right now with this issue: one with ~400 tabs and another with ~150 tabs. One thing to note is the windows in question have been closed for computer shutdown/firefox update and then re-opened. If I open a new window on the same machine, then the issue is not present in that window but still present in the others. So unfortunately it isn't very easy to reproduce.

When I stash a tab in the window with 400 tabs, it will consistently change the active tab to the one 10 tabs to the right. When I stash a tab in the window with 150 tabs, it will consistently change the active tab to the one 5 tabs to the right. If there are fewer than 5/10 tabs to the right, then it will change the active tab to the right-most tab.

This screencap was with Firefox 118.0.1 on MacOS 14.0 using Tab Stash 3.0.

tab_stash.mp4

@josh-berry josh-berry reopened this Oct 29, 2023
@josh-berry josh-berry added i-bug Something might be broken in Tab Stash and removed r-invalid This doesn't seem right labels Oct 29, 2023
@josh-berry
Copy link
Owner

@davidcreel Thanks for sharing the video; I agree that's very strange. It's not clear to me without debugging further what's causing it; my current suspicion is Tab Stash is somehow getting confused about where each tab is in the window, either because it's not tracking this correctly or because Firefox is reporting it incorrectly to Tab Stash. I'll have to see if I can reproduce this on my end to get more info about what's going on.

@bojidar-bg
Copy link

Not sure if it's the same issue or I should open a new one; for me, whenever I stash a tab, I always get the right-most tab selected. Rather annoying when one has to scroll back 20+ tabs to find where one was 😅

@davidcreel
Copy link

davidcreel commented Jun 2, 2024 via email

@AIG420
Copy link

AIG420 commented Sep 9, 2024

I'm having a similar issue. Every time I stash a tab(s) it jumps to the right most tab, no matter how many tabs are in between the stashed tab and the right most one (even 100+).

@bojidar-bg
Copy link

Actually, I think I might have a clue. 👀 When I stash the currently-open tab, the index of the newly-selected tab seems to be off by exactly the sum of pinned and hidden (just-stashed) tabs. Will keep testing that some more, since there might be something else at play too, but that would seem to explain the usual scenario of getting the right-most tab after heavy usage of tab-stash... and it could perhaps even explain the case of different behavior after firefox update/computer shutdown, as firefox might be recreating the hidden tabs then.

@josh-berry
Copy link
Owner

Thanks, that's useful info. I'll poke at this further and see what I can find.

@josh-berry josh-berry added this to the 3.1.1 milestone Sep 22, 2024
@josh-berry
Copy link
Owner

Found the bug, should be fixed in the next release. Noticing that the offset was related to the number of pinned+hidden tabs was the key to finding it. Thanks for reporting and for the help in narrowing it down!

@bojidar-bg
Copy link

Amazing news! Thanks for all the hard work! 💚

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i-bug Something might be broken in Tab Stash
Projects
None yet
Development

No branches or pull requests

5 participants