-
Notifications
You must be signed in to change notification settings - Fork 278
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
[Bug] "Grouping tabs with spaces" snippet crops tab list #3592
Comments
Hmm... sadly the CSS hack is unfriendly to the virtual scrolling architecture of TST 4.0 and later. TST 4.0's virtual scrolling assumes that all tabs have same height and zero vertical margins (except negative margin for collapsed tabs), so virtually scrolled tabs are cut off for margins added by your user stylesheets - it is unavoidable. We may need to implement something new API to register extra margins for tabs and implement a new helper addon which uses the API, instead of the CSS hack. |
I'm afraid that such a new API may make virtual scrolling calculations more slow. Does anyone have something solution idea? |
Thanks for considering the issue. |
@piroor want to move this over to the discussions as this is CSS related? (I did try playing around and thinking about how to solve this and couldn't come up with anything ... I use spacing between my tabs too, but uniformly across all tabs, not just at the root level) |
I have an idea based on blank tabs just for separators, like following:
Something helper addon maintaining such blank tabs for separators between (before) top level tabs and hiding them by CSS may provide the "grouping tabs with spaces" feature virtually. There are some restrictions: the size of spaces for grouping will be same to the height of a tab always, and the space between groups will be clickable unexpectedly so we need to do something for such click events. |
A helper addon I think could certainly facilitate all of this ... within the limitations you mentioned.
The addon could:
Additionally, couldn't you really do this manually with some CSS?
Either way, not sure there is anything more to do on the TST side. :) |
I was trying some CSS to customize appearance.
However, this code snippet induces a bug where the tabs list in TST is cropped at the end.
The more space I use, the more tabs are cropped in TST.
I tried messing with other parameters, and even removing everything in the CSS except from this snippet, and I got the same issue as soon as I enable this snippet.
I tried closing / re-opening the sidebar, and even quitting / relaunching Firefox after applying, still the same.
I have around 2000 tabs, in many groups via "Simple Groups" extension.
The group I was testing on has ~270 opened tabs.
I should mention, all tabs appear correctly in the real tab bar at the top.
Here are some screenshots, with exactly the same tabs opened.
The second one with spaces uses
margin-top: calc(var(--tab-size) / 5);
so that you can see what's missing relative to the "normal" one.Thanks for your help.
Regards.
The text was updated successfully, but these errors were encountered: