Closed Bug 1367712 Opened 3 years ago Closed 3 years ago

Stop using a custom toolbar background color on Windows 7

Categories

(Firefox :: Theme, defect, P1)

Unspecified
Windows 7
defect

Tracking

()

VERIFIED FIXED
Firefox 55
Iteration:
55.6 - May 29
Tracking Status
firefox55 --- fixed
firefox57 --- verified

People

(Reporter: dao, Assigned: dao)

References

(Blocks 1 open bug)

Details

(Whiteboard: [photon-visual][p1][57])

Attachments

(3 files)

Flags: qe-verify+
Priority: -- → P1
Iteration: --- → 55.6 - May 29
QA Contact: brindusa.tot
Comment on attachment 8871229 [details]
Bug 1367712 - Stop using a custom toolbar background color on Windows 7.

https://reviewboard.mozilla.org/r/142726/#review146430

I don't have a Win7 machine with which to test this immediately, but the changes look alright. I'm curious what the final effect of this patch is, can you post a screenshot? :)
Attachment #8871229 - Flags: review?(nhnt11) → review+
Attached image screenshot
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8dc08e2b866b
Stop using a custom toolbar background color on Windows 7. r=nhnt11
https://hg.mozilla.org/mozilla-central/rev/8dc08e2b866b
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Depends on: 1368503
I tested this issue on Windows 7 with the latest Nightly 57.0a1(2017-08-09) and I have different results from the one presented in Screenshots. Please see the attached file.
Flags: needinfo?(dao+bmo)
(In reply to ovidiu boca[:Ovidiu] from comment #7)
> Created attachment 8895364 [details]
> comparing Nightly with screenshoots.png
> 
> I tested this issue on Windows 7 with the latest Nightly 57.0a1(2017-08-09)
> and I have different results from the one presented in Screenshots. Please
> see the attached file.

The important part is that we're not using the blue toolbar color anymore.
Flags: needinfo?(dao+bmo)
Thanks Dão, based on comment 8 I will mark this as verified fixed.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.