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

The wiki suggestion to use "dynamic native tabs" using userChrome.css doesn't work with firefox 133 #1889

Open
ricardoseriani opened this issue Nov 27, 2024 · 4 comments

Comments

@ricardoseriani
Copy link

Steps to reproduce

Actual behavior

When you open firefox 133 the tabs are not hidding when Sidebery is opened

Expected behavior

Firefox tabs should be hide when Sidebery is open.
After some research, I found that the #titlebar "selector" is removed from this version.
I tried to update the userChrome.css and get to a new version that is working for me, but I can't change or submit a PR to change the wiki.
All that it's need is to update the selector to .toolbar-items like the following snippet.

#main-window .toolbar-items {
    overflow: hidden;
    transition: height 0.3s 0.3s !important;
}

/* Default state: Set initial height to enable animation */
#main-window .toolbar-items {
    height: 3em !important;
}

#main-window[uidensity="touch"] .toolbar-items {
    height: 3.35em !important;
}

#main-window[uidensity="compact"] .toolbar-items {
    height: 2.7em !important;
}

/* Hidden state: Hide native tabs strip */
#main-window[titlepreface*="[Sidebery]"] .toolbar-items {
    height: 0 !important;
}

/* Hidden state: Fix z-index of active pinned tabs */
#main-window[titlepreface*="[Sidebery]"] #tabbrowser-tabs {
    z-index: 0 !important;
}

Please, note that I only tested on my machine and I'm no expert in CSS.

System

Xubuntu 22.04

Firefox version

133.0

Sidebery version

5.2.0

Logs

No response

@cem-okulmus
Copy link

cem-okulmus commented Nov 28, 2024

Even when using this exact userChrome snippet, I still see Firefox bringing up the native tabs. It even shows Sideberry as a "sidebar extension", so this looks like it is almost hard-coded behaviour to have them both visible (which is pretty strange, clearly the point should be that Firefox hands off things to the dedicated "sidebar extension" then). I would suspect this is a bug.

I should perhaps mention that I am using the "vertical tabs" option inside Firefox, as that also fixes things like window controls. Under horizontal tabs (and with sidebar.revamp set to false), I think this fix works.

@oktayacikalin
Copy link

oktayacikalin commented Nov 28, 2024

I'm using horizontal tabs with sidebery. That fix above works fine in 133 using Gnome.

I'm also using the firefox-gnome-theme and setting the sizes to 2.7em, 3.05em and 2.4em seems to fit better.

@cem-okulmus
Copy link

I'm using horizontal tabs with sidebery. That fix above works fine in 133 using Gnome.

I'm also using the firefox-gnome-theme and setting the sizes to 2.7em, 3.05em and 2.4em seems to fit better.

Yeah, I decided to switch to horizontal tabs. Long-term, I think now that Firefox does support vertical stuff natively, perhaps Firefox could just allow extensions to swap in their own side bars instead of the native vertical bar, which would be the best of both world, where you get custom sidebar extensions without having to mess with userChrome at all.

@ricardoseriani
Copy link
Author

I'm using horizontal tabs with sidebery. That fix above works fine in 133 using Gnome.
I'm also using the firefox-gnome-theme and setting the sizes to 2.7em, 3.05em and 2.4em seems to fit better.

Yeah, I decided to switch to horizontal tabs. Long-term, I think now that Firefox does support vertical stuff natively, perhaps Firefox could just allow extensions to swap in their own side bars instead of the native vertical bar, which would be the best of both world, where you get custom sidebar extensions without having to mess with userChrome at all.

Please, although I have not mentioned, be aware that you have to set the configuration preface value to [Sidebery] inside Sidebery configuration -> General.
By the way, my Firefox doesn't have an option to enable vertical support for tabs in the current release, only in the nightly release.
But I agree that will be awesome if Firefox allow extension to swap with the native bar.

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

No branches or pull requests

3 participants