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

Windows 7 minimize maximize close buttons hidden #218

Closed
3 of 13 tasks
picollo7 opened this issue Sep 12, 2021 · 15 comments
Closed
3 of 13 tasks

Windows 7 minimize maximize close buttons hidden #218

picollo7 opened this issue Sep 12, 2021 · 15 comments
Labels
Class::Color Color, background, blur, etc. Class::SideEffect Unexpected side effect, undefined behavior Component::Toolbar Toolbar or navbar Env::Win7 Issues on Windows7 Issue::Bug Something isn't working Priority::High Solve and focus on it first

Comments

@picollo7
Copy link

Describe the bug

In Windows 7 the minimize, maximize, and close buttons are covered by the background image in dark mode. They still work, but are not visible.

Expected behavior

The minimize, maximize, and close buttons should be visible.

Screenshots

image

Environment:

Additional context

@picollo7 picollo7 added the Issue::Bug Something isn't working label Sep 12, 2021
@black7375 black7375 added Env::Windows Issues on Windows10, Windows11, unknown windows Priority::High Solve and focus on it first Class::Color Color, background, blur, etc. Component::Toolbar Toolbar or navbar Class::SideEffect Unexpected side effect, undefined behavior labels Sep 12, 2021
@black7375
Copy link
Owner

Would you like to upgrade to a lastest commit?

@jskm
Copy link

jskm commented Sep 19, 2021

This issue is caused by changes in Firefox. I see it with Firefox 92 running on Windows 7 with Aero.

I tried earlier versions of Lepton until v1.0 before getting the titlebar buttons visible again. It is d16cd6d that makes the navigator-toolbox opaque and ends up blocking the titlebar buttons, but I've been using much newer versions than that without problems, so the difference had to be a Firefox update.

I downloaded Firefox 90, 91, and the 91.1 ESR, and narrowed down the change to sometime after 91.0.2. In fact, even unaltered, Firefox 92 and 91.1.0esr paint over the title bar buttons as shown in the second image here:

Firefox 91.0.2:

Firefox-91 0 2

Firefox 91.1.0esr:

Firefox-91 1 0esr

While toggling properties in the Browser Toolbox, I found that everything went back to normal, even having Aero transparency in the title bar again, after removing this setting introduced in d16cd6d:

:root:not([lwtheme-image]) #navigator-toolbox:-moz-lwtheme {
background-color: var(--lwt-accent-color) !important;
}

Lepton, fixed:

Firefox-ESR-Lepton

It seems #TabsToolbar { background-image: unset !important } is what removes the broken coloration of the tab bar seen in default Firefox. To change the background of the tab bar as before, you'd have to be able to cut out a transparent region over the title bar buttons.

@black7375
Copy link
Owner

black7375 commented Sep 19, 2021

Based on the latest commit, the symptom does not appear at my VM.
image

@jskm
Copy link

jskm commented Sep 19, 2021

I downloaded the portable version of Firefox 92, ran it, and then installed the latest commit for Lepton. Without Lepton, the tab bar has the white overlay painted over the title bar buttons, as shown in my previous screenshot. With Lepton, it looks correct with Aero transparency and visible system buttons, until I go to Customize Toolbar -> Themes and choose the Light or Dark theme. Then the system buttons are painted over completely.

It turns out that the "System theme" setting works correctly, but I've never used it because it is light-themed on Windows 7. The Firefox Dark theme still does not seem to work, showing the problem as described in the first post. I've just discovered the ui.systemUsesDarkTheme option in combination with the Firefox System theme which does work as intended. Though it seems there's transparency if ui.systemUsesDarkTheme is set for light mode, and solid color when set to dark.

@black7375
Copy link
Owner

Looks like I found a workaround yesterday.
https://www.reddit.com/r/waterfox/comments/qk0rjz/waterfox_g4_themes/hjakxzs/?context=3

Download ff_win7_darkmode.reg and double-clicking on it.

Does it works?

@Kiki79250CoC
Copy link

OK. This reg fix partially fixes the problem. Let me explain.

  • When I use the System theme, It works (Image 1)
  • When I use one of the newest theme introduced with Firefox 94, it Works (Image 2 & 3)
  • But When I use the "full" Dark and Light theme, it doesn't work (Image 4 & 5).

(Image 1 :)
image

(Image 2 :)
image

Image 3 :)
image

(Image 4 :)
image

(Image 5 :)
image

I remember that before Firefox 89, only tabs back have the color, leaving the whole window have the Aero (Here in Firefox 90.0.2 with Ptoton Disabled) :
image

@black7375
Copy link
Owner

@Kiki79250CoC

Thank you very much for doing a lot of testing.
Can you attach the about:support - Copy text to clipboard information for reproduction?

@Kiki79250CoC
Copy link

Here is the content of the clipboard after clicking the button :
(if you can't find the info I snapped the whole page here : about config page.zip)

  <h2 xmlns="http://www.w3.org/1999/xhtml" class="major-section" data-l10n-id="app-basics-title">Paramètres de base de l’application</h2>
Nom Firefox
94.0.2
20211119140621

release
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:94.0) Gecko/20100101 Firefox/94.0
Windows_NT 6.1 7601
Activé
1/1
0/1 Désactivées par défaut
11
Inactives
Présente
Présente
Présente
false

<!--EndFragment-->

@black7375
Copy link
Owner

I just tried it, and the pasting comes out weird.
I'm sorry, but can you do it again with Copy raw data to clipboard?

@Kiki79250CoC
Copy link

I put the raw data in a text file :
Raw data.txt

black7375 added a commit that referenced this issue Dec 1, 2021
I'am not sure as it's not reproducible.
@black7375
Copy link
Owner

Reproducibility is quite difficult.
Win7-2021-12-01-12-07-43

However, once I applied a patch that could solve it. 8a8351d
I'm not sure if it can be solved. A test is required.

@Kiki79250CoC
Copy link

Kiki79250CoC commented Dec 1, 2021

OK. So it didn't fixed the problem, BUT I think I know why...

On those two color schemes, the default Light and Dark Themes have the white "gradient" which overlaps the window frame (and the buttons with) :
image

And when I use an alternative theme (like Alpenglow or one of the themes introduced with Firefox 94), the overlapping is not present at all :
image
image

So I think this problem is not from Lepton, but from Firefox itself.
EDIT : I found the BugZilla ticket concerning this problem.


For the moment I can suggest you a workaround.
Coloring only the back of inactive tabs like original Firefox 88 and completly remove the back calque like the System Theme. (Image 1 of the previous post)

@black7375
Copy link
Owner

I followed the color-way renders.
This time, I predict that the symptoms will probably go away.


I was working on following the default theme of Win7, but found that it was not applied in color-way.

If an option system is adopted in the future, we are going to add an option to forcibly activate the Win7 default like theme.
(Just for memo)

/*= Windows 7, 8 - Tab Bar Background *****************************************/
@media (-moz-os-version: windows-win7) {
  /* Header Color */
  :root:not([lwtheme-image]):-moz-lwtheme #menubar-items,
  :root:not([lwtheme-image]):-moz-lwtheme .tab-background:not([selected], [multiselected]),
  :root:not([lwtheme-image]):-moz-lwtheme #titlebar .toolbarbutton-1 > .toolbarbutton-icon,
  :root:not([lwtheme-image]):-moz-lwtheme #alltabs-button > .toolbarbutton-badge-stack,
  #scrollbutton-up:-moz-lwtheme,
  #scrollbutton-down:-moz-lwtheme {
    background-color: var(--lwt-accent-color) !important;
  }

  :root:not([lwtheme-image]):-moz-lwtheme .tabbrowser-tab:hover > .tab-stack > .tab-background:not([selected], [multiselected]),
  :root:not([lwtheme-image]):-moz-lwtheme #titlebar .toolbarbutton-1:hover > .toolbarbutton-icon,
  :root:not([lwtheme-image]):-moz-lwtheme #alltabs-button:hover > .toolbarbutton-badge-stack,
  #scrollbutton-up:-moz-lwtheme:hover,
  #scrollbutton-down:-moz-lwtheme:hover {
    background-color: color-mix(in srgb, currentColor 15%, var(--lwt-accent-color)) !important;
  }
  :root:not([lwtheme-image]):-moz-lwtheme #titlebar .toolbarbutton-1:is([open], :hover:active) > .toolbarbutton-icon,
  :root:not([lwtheme-image]):-moz-lwtheme #alltabs-button:is([open], :hover:active) > .toolbarbutton-badge-stack,
  #scrollbutton-up:-moz-lwtheme:hover:active,
  #scrollbutton-down:-moz-lwtheme:hover:active {
    background-color: color-mix(in srgb, currentColor 20%, var(--lwt-accent-color)) !important;
  }

  #TabsToolbar:-moz-lwtheme {
    color: var(--lwt-text-color) !important;
  }

  /* Remove Aero */
  #TabsToolbar {
    /* Original: radial-gradient(eclipse at bottom, rgba(255, 255, 255, 0.8), rgba(255, 255, 255, 0.5) 80%, transparent) */
    background-image: unset !important;
  }
}
@media (-moz-os-version: windows-win8) {
  /* Header Color */
  :root:not([lwtheme-image]):-moz-lwtheme {
    background-color: var(--lwt-accent-color) !important;
  }

  #navigator-toolbox:-moz-lwtheme {
    color: var(--lwt-text-color) !important;
  }
}

@Kiki79250CoC
Copy link

Installed the updated version, and it seems to be finally fixed! (Firefox 95)

Light Theme :
image

Dark Theme :
image

@black7375
Copy link
Owner

@Kiki79250CoC
Thank you very much for participating in the issue and testing every time!!

@black7375 black7375 added Env::Win7 Issues on Windows7 and removed Env::Windows Issues on Windows10, Windows11, unknown windows labels Apr 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Class::Color Color, background, blur, etc. Class::SideEffect Unexpected side effect, undefined behavior Component::Toolbar Toolbar or navbar Env::Win7 Issues on Windows7 Issue::Bug Something isn't working Priority::High Solve and focus on it first
Projects
None yet
Development

No branches or pull requests

4 participants