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

Tabs + window controls not displayed with v5 default settings #368

Closed
3 of 13 tasks
redgoldlace opened this issue Apr 26, 2022 · 6 comments
Closed
3 of 13 tasks

Tabs + window controls not displayed with v5 default settings #368

redgoldlace opened this issue Apr 26, 2022 · 6 comments
Labels
Class::Layout-Others Layout issues except Padding and Size issues Class::SideEffect Unexpected side effect, undefined behavior Component::Tabbar Tab or tabbar Env::Windows Issues on Windows10, Windows11, unknown windows Issue::Bug Something isn't working Priority::High Solve and focus on it first

Comments

@redgoldlace
Copy link

Describe the bug
I think this is probably pretty self-explanatory, but if you don't enable userChrome.tab.on_bottom you.. just won't
have tabs or window controls at all. Needless to say that makes navigation a bit difficult:
An image depicting a rather headless Firefox instance

And no. This image is not cropped.

I did some hunting in leptonChrome.css and seem to have found the culprit. Applying the following CSS
unconditionally (i.e, not within a media query) appears to mostly fix the issue:

#titlebar {
  -moz-appearance: none !important;
}

There were a few other weird tweaks i had to make, and for the sake of completeness I'll leave them here if it's helpful
in providing a solution:

Expand codeblock
#navigator-toolbox {
  display: flex !important;
  flex-wrap: wrap;
}

#titlebar,
#nav-bar,
#PersonalToolbar,
#tab-notification-deck,
#tab-notification-deck-template {
  flex-basis: 100%;
}

#titlebar {
  order: 0;
  -moz-appearance: none !important;
  --tabs-navbar-shadow-size: 0px;
}

#tab-notification-deck {
  order: 1;
}

#PersonalToolbar {
  order: 2;
}

#navigator-toolbox {
  padding-top: var(--uc-titlebar-padding) !important;
}

:root:not([sizemode="maximized"]) {
  --uc-titlebar-padding: 0px;
}

:root[sizemode="maximized"] {
  --uc-titlebar-padding: 8px;
}

I'm not sure how many of these changes are necessary. I used the existing CSS and good ol' trial and error until
Firefox behaved. I opened an issue instead of a PR since I... honestly have no idea what I'm doing. This is my first
time interacting with Lepton or Firefox in this way, so I figure somebody else will have a better solution to this.

I really appreciate the project and couldn't live without it. Thanks for your hard work!

Environment:

@redgoldlace redgoldlace added the Issue::Bug Something isn't working label Apr 26, 2022
@black7375 black7375 added Env::Windows Issues on Windows10, Windows11, unknown windows Component::Tabbar Tab or tabbar Class::Layout-Others Layout issues except Padding and Size issues labels Apr 26, 2022
@black7375
Copy link
Owner

It works well on Linux, but I'll check Windows. Is it Windows 10?
image

@redgoldlace
Copy link
Author

It is Windows 10, yeah. I notice in your screenshot that the tabs are displayed on the bottom - I'm referring to not enabling that setting. That's when I have no tabs and no window controls at all.

For context, this is (roughly) how I would expect tabs to look with userChrome.tab.on_bottom set to false:
How I would expect tabs to look by default

@erwin4270
Copy link

erwin4270 commented Apr 26, 2022

If you view/enable the title bar, the tab bar and window controls will show. (temporary workaround for me)
Title bar disabled:
4
Title bar enabled:
5

@117649
Copy link

117649 commented Apr 26, 2022

@black7375 @kaylynn234
Same here.
And even worse:
image
Because I have my bookmarks bar auto hide and over page it now not display at under navbar but at top of the window and over everything.

All fixes codes promoted have been applied.
Switch back to 4.6.3 for now.

@black7375 black7375 added the Priority::High Solve and focus on it first label Apr 26, 2022
black7375 added a commit that referenced this issue Apr 26, 2022
black7375 added a commit that referenced this issue Apr 26, 2022
@black7375
Copy link
Owner

black7375 commented Apr 26, 2022

I created a new release.

Tested with Win7 VM, WIn10 VM, Mac VM

@s-rog
Copy link

s-rog commented Apr 26, 2022

windowed:
Capture
maximized:
Capture1
@black7375 The top of tab/title bar is cut off when maximized with this new version :/ (but at least it showed up!)

@black7375 black7375 added the Class::SideEffect Unexpected side effect, undefined behavior label Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Class::Layout-Others Layout issues except Padding and Size issues Class::SideEffect Unexpected side effect, undefined behavior Component::Tabbar Tab or tabbar Env::Windows Issues on Windows10, Windows11, unknown windows Issue::Bug Something isn't working Priority::High Solve and focus on it first
Projects
None yet
Development

No branches or pull requests

5 participants