-
Notifications
You must be signed in to change notification settings - Fork 280
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
Grouping tabs without indentation, but separating groups by white space #2777
Comments
Spacings between root level tabs can be emulated with a custom style rules like: /* show separator space between root level tabs */
tab-item[data-level="0"] {
margin-top: var(--tab-size);
}
/* cancel needless separators on exceptions */
tab-item[data-level="0"].pinned,
tab-item[data-level="0"].pinned + tab-item[data-level="0"],
tab-item[data-level="0"] + tab-item[data-level="0"]:not([data-child-ids]) {
margin-top: 0;
}
/* make collapsed tabs visible */
tab-item:not(.pinned).collapsed,
tab-item:not(.pinned).collapsed.collapsed-completely {
margin-top: 0;
opacity: 1;
visibility: visible;
} Setting the max indent level to zero with this style rules will work like the old one. |
Sorry but it won't be a built-in option of TST itself, because:
|
Jesus Christ, man, since webextensions I suffering from lack of this feature! It works like a charm. Thanks. I modified it a little bit for my taste: /* show separator space between root level tabs */
tab-item[data-level="0"] {
margin-top: 8px;
}
/* cancel needless separators on exceptions */
tab-item[data-level="0"].pinned,
tab-item[data-level="0"].pinned + tab-item[data-level="0"],
tab-item[data-level="0"] + tab-item[data-level="0"]:not([data-child-ids]) {
margin-top: 0;
}
/* sidebar background */
:root.sidebar #background {
background: #6c6c6c;
} Maybe you should add it to the "Code Snippets" documentation. |
@icher, can you close this item? Thanks. |
interestingly, i get a 'ghost' tab in the background of my side bar if i use @icher 's sidebar background color CSS, but only if i replace it with a gradient, such as /* sidebar background */
:root.sidebar #background {
background: rgb(238,156,167) linear-gradient(90deg, rgba(238,156,167,1) 0%, rgba(251,193,200,1) 100%);
opacity: .25 !important;
} |
I've added a new section to the code snippets: |
Actually, last section "make collapsed tabs visible" do not need, collapse group is useful feature. Without this section all works great. The idea is just economy of space, no change the behaviour. |
OK, I've removed the last section. |
@piroor do you have any idea what the 'thing' in my screenshot could be? |
Ah, it is by design. TST has a "fake" tab behind the background screen element to calculate size of tabs, favicons, and others. It will be revealed unexpectedly for now, if you use transparent color for the |
seems like i peeked behind the wizard's curtain by accident :D appreciate the explanation |
The commit 9411efb should hide the dummy elements completely. |
amazing work you're doing here, thank you once again!! |
@piroor hmm seems like you managed to get rid of the dummy tab, but these 'open new tab as' (?) options still linger |
well they're only present briefly when you activate the sidebar, so i guess they're not a big deal |
@mimecry Sorry I missed that. Here is a workaround: body > ul[id$="selector"] {
opacity: 0;
} Now it is included by default with the commit 61a6fbb. |
Hi.
Old TST (pre-webextensions) allowed grouping tabs without indentation, but separated by white space. May be it was undocumented function or even a bug, only worked by editing about:config variable, some kind of "indent_tabs_until=-1" or like that. It was VERY useful. But new TST can't do that. Is it possible to implement that behavior by webextension-api?
Bookmarks sidebar can use separators ("Add separator" horizontal straight line) -- may be it's the way to go.
The text was updated successfully, but these errors were encountered: