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

[Bug] changing --tab-size breaks tab drag #3519

Closed
vitaliylag opened this issue Apr 4, 2024 · 3 comments
Closed

[Bug] changing --tab-size breaks tab drag #3519

vitaliylag opened this issue Apr 4, 2024 · 3 comments

Comments

@vitaliylag
Copy link

vitaliylag commented Apr 4, 2024

Abstract

Steps to reproduce

  1. Start Firefox with clean profile.
  2. Install TST.
  3. Use this style:
:root {
	--tab-size: 28px;
}
  1. Open at least few tabs. The more tabs you open the bigger difference between expected cumulative tab size and cumulative --tab-size. If the difference is not enough the bug will not occur.
  2. Try to drag the last tab to previous tab.

Expected result

The size is changed and all other functions work fine.

Actual result

The size is changed but when you try to drag one tab to another you cannot create a new subtree. So if target tab does not have children the source tab cannot become the child of target tab.

Also even if target tab already has children it's become impossible to move the tab to the end of it's child list. It's possible to move to any other position though so you can move source tab to the position before last child and then swap last 2 children but it's not convenient.

It worked fine before last big update.

Environment

  • Platform (OS): Windows 11
  • Version of Firefox: 124.0.1
  • Version (or revision) of Tree Style Tab: 4.0.10

Possible fast way to fix

Add "tab-size" option to add-on settings. It probably will fix the bug (because the value in Javascript will match the css value) and it also will be convenient for users.

@piroor
Copy link
Owner

piroor commented Apr 4, 2024

Thanks, the commit bd24eb6 should fix this. Here is a workaround user style sheet:

#dummy-tabs tab-item {
  height: var(--tab-size);
}

@piroor
Copy link
Owner

piroor commented Apr 5, 2024

Updated: I've introduced more changes around tab size calculation priority. The workaround in my previous comment need to be removed after a new release containing those changes is released.

Copy link

This issue has been closed due to no response within 14 days after labeled as "fixed", 7 days after last reopened, and 7 days after last commented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants