Closed Bug 1700873 Opened 3 years ago Closed 3 years ago

[win7] White/Dark line covering the buttons over the tab bar when Dark/Light theme is used an browser is unfocussed

Categories

(Firefox :: Theme, defect)

Desktop
Windows 7
defect
Points:
2

Tracking

()

VERIFIED FIXED
89 Branch
Iteration:
89.1 - Mar 22 - Apr 4
Tracking Status
firefox-esr78 --- unaffected
firefox86 --- unaffected
firefox87 --- unaffected
firefox88 --- verified
firefox89 --- verified

People

(Reporter: atrif, Assigned: bugzilla)

References

(Regression)

Details

(Keywords: regression)

Attachments

(5 files)

Attached image button.gif

The issue has been seen here by Simon Mainey.

Affected versions

  • 89.0a1 (20210324084848)
  • 88.0b2 (20210323190052)

Affected platforms

  • Windows 7x64

Steps to reproduce

  1. Open firefox set Dark/ Light theme.
  2. Unfocus the browser.
  3. Observe the top part of the browser.

Expected result

  • No glitches are displayed.

Actual result

  • Black/White line covering the buttons.

Regression range

  • With the build before bug 1694734 the issue is immediately visible when setting a Dark/Light or a custom theme. After bug 1694734 the issue is visible on the Light/Dark theme when the browser is not focused.

Notes

  • Attached a screen recording.
  • With webrender disabled the line is displayed but is not covering the Minimize, Close, and Maximize buttons.
  • The issue is reproducible on Firefox 89.0a1 (20210324160546) with proton as well if Dark/Light themes are used (not Proton Dark/Light themes).

Hi Harry! Can you please have another look at this? Thank you!

Has Regression Range: --- → yes
Has STR: --- → yes
Flags: needinfo?(htwyford)
Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Iteration: --- → 89.1 - Mar 22 - Apr 4
Points: --- → 2
Flags: needinfo?(htwyford)

This regression was partially fixed in D106670. This sets the remaining window background properties on :root in Windows 7/8.

Pushed by htwyford@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0ce7f89cbb8a
Set all window background properties on :root instead of #navigator-toolbox on Windows 7/8. r=desktop-theme-reviewers,dao
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

Backed out for causing light text on light background with Alpenglow theme.

Backout link: https://hg.mozilla.org/integration/autoland/rev/d4dfcdef9bf88b6edc169dd1b26d9b31a8684c47

Status: RESOLVED → REOPENED
Flags: needinfo?(htwyford)
Resolution: FIXED → ---
Target Milestone: 89 Branch → ---
Backout by abutkovits@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/544196bf45f8
Backed out changeset 0ce7f89cbb8a for causing light text on light background with Alpenglow theme. CLOSED TREE
Flags: needinfo?(htwyford) → needinfo?(alexandru.trif)

I've tried the test build on a fresh profile on Windows 10 and it shows the white background in the tab bar again, unfortunately.

Sebastian

Furthermore, I can also see the white line on the Dark theme on Windows 10. (Though there is no dark line on the Light theme.

Sebastian

Thanks for trying the build. I should have clarified that I didn't expect that build to work on Windows 10. I noticed a Windows 10 issue on it shortly after I posted it, but I didn't update the try run since I was looking for Windows 7/8 feedback.

Could you please try this build? It should have the correct behaviour on all versions of Windows.

https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/bDor1K_FRBKba3wt7uuaFA/runs/0/artifacts/public/build/install/sea/target.installer.exe

Treeherder: https://treeherder.mozilla.org/jobs?repo=try&revision=94cd87001f812e7c30d1c7cf25ccad9b7672cfd5&selectedTaskRun=bDor1K_FRBKba3wt7uuaFA.0

Hi Harry! I can confirm that using the build from comment 11 on Windows 7x64 the tab bar is properly displayed with Light/ Dark/ Alpenglow and a random theme when applied and when the browser is not focused.

On Windows 10x64 I saw that the white line is still visible in with Dark theme and Alpenglow theme when the browser is not focused. But there are two scenarios:

Also, I cannot reproduce bug 1701436 with the provided build on Windows 10x64.

Flags: needinfo?(alexandru.trif)

Thanks. I can reproduce the white on today's Nightly on Windows 10, but I can't reproduce it with this patch applied on the latest central. I think that may be because the latest central includes Proton tabs by default, which change the composition of the tabs bar. Since that line isn't a regression from bug 1694734, I'm going to land this patch. If the white line is an issue in tomorrow's Nightly, I'll file a new bug and address it there.

Pushed by htwyford@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8d64728a5c36
Set all window background properties on :root instead of #navigator-toolbox on Windows 7/8. r=desktop-theme-reviewers,dao
Status: REOPENED → RESOLVED
Closed: 3 years ago3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

Comment on attachment 9211628 [details]
Bug 1700873 - Set all window background properties on :root instead of #navigator-toolbox on Windows 7/8. r?#desktop-theme-reviewers!

Beta/Release Uplift Approval Request

  • User impact if declined: Fixes theme regression on windows 7/8 that hides the close/min/max buttons.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce: Build already tested by QA.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): CSS-only change covered by tests.
  • String changes made/needed:
Attachment #9211628 - Flags: approval-mozilla-beta?

Comment on attachment 9211628 [details]
Bug 1700873 - Set all window background properties on :root instead of #navigator-toolbox on Windows 7/8. r?#desktop-theme-reviewers!

Thanks for adding more test coverage. Approved for 88.0b6.

Attachment #9211628 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attached image white_beta.gif

Hello! Verified on Windows 7x64 and Windows 10x64 with Firefox 89.0a1 (20210405094633) and 88.0b7 (20210404185932) and there are
the results:
On windows7x64 - the issue is fixed - the Tab bar is properly displayed alongside with Close, Minimize and Maximize buttons when a random or default theme is selected and the browser is focused/unfocused.

On windows 10x64 - the white line is no longer displayed with Firefox 89.0a1 but unfortunately is displayed with 88.0b7 while using Dark theme. Harry, should we close this bug as verified and open another one for the remaining one on Firefox beta 88? Thank you!

Flags: needinfo?(htwyford)

Yes, please file a new bug for the Windows 10 88 beta issue. Thanks!

Flags: needinfo?(htwyford) → needinfo?(alexandru.trif)
See Also: → 1703546

Thank you, Harry. Filled bug 1703546. Closing this as verified per comment 19 and comment 20.

Status: RESOLVED → VERIFIED
Flags: needinfo?(alexandru.trif)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: