Auto-scroll to active tab doesn't always work - new

[chaoscreater]

Hi,

Not sure if you saw my latest comment in this issue that I created a while ago, but the problem still persists in v3.4.6.

#73

To reproduce the problem:

1. Click Active Tabs panel.
2. Choose a tab at the very bottom (e.g. tab 100).
3. Click All Tabs panel.
4. Scroll position is still at e.g. tab 10, not tab 100

Another way to reproduce the problem:

1. Click tab 10
2. In the search bar, search for tab 100. Select it.
3. Cancel/remove the search. It should jump to tab 100, but the scroll position is still at tab 10.

[ymlluo]

Thanks for the feedback. This was indeed an oversight regarding the tab panel switching scenario. We will submit a new version this week to fix this issue.

[chaoscreater]

Actually, I just realized that when you search for a tab and then close the search, it doesn't jump to the active tab if you have lots of tab groups. If I test this without tab groups and I just have maybe 50 tabs open, then it seems to work OK.

I'm using v3.4.7, the latest as of this writing.

I just tried using Claude to generate a fix for me and it seems to have worked. I've attached the fix for your reference.

searchScrollFix.js

Then just edit sidebar and sidepanel HTML files to reference it:

<script defer="defer" src="/searchScrollFix.js"></script>

[ymlluo]

Thanks for your feedback.

Since scrolling is based on DOM elements, it may cause anomalies if the DOM element does not exist (for example, when rendering search results but not tabs during a search).

I will refer to your code to continue checking and reproducing/fixing this issue.