Color the title bar by default on Windows 10

VERIFIED FIXED in Firefox 55

Status

()

defect
P1
normal
VERIFIED FIXED
2 years ago
4 months ago

People

(Reporter: dao, Assigned: dao)

Tracking

(Depends on 1 bug, Blocks 2 bugs)

Trunk
Firefox 55
All
Windows 10
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 verified)

Details

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

Attachments

(1 attachment)

(Assignee)

Updated

2 years ago
Priority: -- → P2
Whiteboard: [photon-visual][p1][57] → [photon-visual][p1][57] [triage]
QA Contact: brindusa.tot
Whiteboard: [photon-visual][p1][57] [triage] → [photon-visual][p1][57]
(Assignee)

Updated

2 years ago
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Priority: P2 → P1
Iteration: --- → 55.6 - May 29
(Assignee)

Comment 2

2 years ago
Sorry for the somewhat messy patch. I could have used caption-buttons.svg#*-white to adjust to the new title bar color, but I thought now would be a good time for some refactoring and then things got a bit out of control...
(Assignee)

Updated

2 years ago
Blocks: 1358998

Comment 3

2 years ago
mozreview-review
Comment on attachment 8871892 [details]
Bug 1367384 - Make the title bar dark by default on Windows 10 and refactor caption button icons.

https://reviewboard.mozilla.org/r/143424/#review147592

Great, thanks! Just a couple of minor things.

::: browser/themes/windows/browser-aero.css:119
(Diff revision 1)
> -      #titlebar-close:hover {
> -        list-style-image: url(chrome://browser/skin/caption-buttons.svg#close-white);
>        }
>  
> +      .titlebar-button:-moz-lwtheme {
> +        -moz-context-properties: unset;

This doesn't really have any effect since the stroke values are hardcoded anyway, right? Any reason for keeping this rule around?

::: browser/themes/windows/window-controls/maximize-highcontrast.svg:4
(Diff revision 1)
> +<!-- This Source Code Form is subject to the terms of the Mozilla Public
> +   - License, v. 2.0. If a copy of the MPL was not distributed with this
> +   - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
> +<svg width="12" height="12" xmlns="http://www.w3.org/2000/svg">

Missing a shape-rendering="crispEdges"? Assuming that it's still necessary.

::: browser/themes/windows/window-controls/maximize.svg:4
(Diff revision 1)
> +<!-- This Source Code Form is subject to the terms of the Mozilla Public
> +   - License, v. 2.0. If a copy of the MPL was not distributed with this
> +   - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
> +<svg width="12" height="12" xmlns="http://www.w3.org/2000/svg">

shape-rendering="crispEdges"?

::: browser/themes/windows/window-controls/restore-themes.svg:7
(Diff revision 1)
> +   - License, v. 2.0. If a copy of the MPL was not distributed with this
> +   - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
> +<svg width="12" height="12" xmlns="http://www.w3.org/2000/svg" shape-rendering="crispEdges" fill="none" stroke="white">
> +  <path stroke="black" stroke-width="3.6" stroke-opacity=".75" d="M2,4 l 6,0 l 0,6 l -6,0z M2.5,1.5 l 8,0 l 0,8"/>
> +  <rect stroke-width="1.9" x="2" y="4" width="6" height="6"/>
> +  <polyline stroke-width=".9" points="3.5,1.5 10.5,1.5 10.5,8.5" class="restore-background-window"/>

I think you can remove class=restore-background-window
Attachment #8871892 - Flags: review?(jhofmann) → review+
(Assignee)

Comment 4

2 years ago
(In reply to Johann Hofmann [:johannh] from comment #3)
> ::: browser/themes/windows/browser-aero.css:119
> (Diff revision 1)
> > -      #titlebar-close:hover {
> > -        list-style-image: url(chrome://browser/skin/caption-buttons.svg#close-white);
> >        }
> >  
> > +      .titlebar-button:-moz-lwtheme {
> > +        -moz-context-properties: unset;
> 
> This doesn't really have any effect since the stroke values are hardcoded
> anyway, right? Any reason for keeping this rule around?

Yes, it tells Gecko that it doesn't need to bother forwarding context properties in the first place. It's supposed to be cheaper.
Comment hidden (mozreview-request)

Comment 6

2 years ago
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f9541bc8ef95
Make the title bar dark by default on Windows 10 and refactor caption button icons. r=johannh

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f9541bc8ef95
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55

Updated

2 years ago
Depends on: 1368672

Updated

2 years ago
Depends on: 1368679
Depends on: 1368808
I have seen the feature being implemented with latest Nightly 55.0a1 on Windows 10 (64 bit).

This bug's fix is now verified in Latest Nightly.

Build ID   :   20170531030204
User Agent :   Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:55.0) Gecko/20100101 Firefox/55.0

[bugday 20170531]

Updated

2 years ago
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Depends on: 1372689

Updated

a year ago
Depends on: 1428368

Updated

4 months ago
No longer depends on: 1515714
You need to log in before you can comment on or make changes to this bug.