Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-02-17 22:32:51 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1671252 - Move Windows accent border to #navigator-toolbox. r=Gijs

Browse Source 
This works correctly. On the root it seems we hit this codepath:

  https://searchfox.org/mozilla-central/rev/df18dd52da04ee2bad434b0ba2d9fcb196d4d15e/layout/xul/nsBoxFrame.cpp#514-525

And end up with an slightly-bigger-than-needed box. I haven't dug into what
changing this may entail, but given the root frame is special anyways (because
it affects window constraints etc) it's probably safer to do this.

Differential Revision: https://phabricator.services.mozilla.com/D131421
This commit is contained in:
Emilio Cobos Alvarez 2021-11-17 18:30:17 +00:00
parent 868db9824e
commit dddeefe891
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
browser/themes/windows/browser-aero.css
View File

@ -45,12 +45,12 @@
@media not (-moz-os-version: windows-win8) {
@media (-moz-windows-default-theme) {
@media (-moz-windows-accent-color-in-titlebar) {
:root[sizemode=normal][tabsintitlebar] {
:root[sizemode=normal][tabsintitlebar] #navigator-toolbox {
border-top: .5px solid -moz-accent-color;
}
}
:root[sizemode=normal][tabsintitlebar]:-moz-window-inactive {
:root[sizemode=normal][tabsintitlebar]:-moz-window-inactive #navigator-toolbox {
border-top-color: rgb(57,57,57);
}
@ -267,7 +267,8 @@
@media (-moz-os-version: windows-win8) {
/* Artificially draw window borders that are covered by lwtheme, see bug 591930.
* Borders for win7 are below, win10 doesn't need them. */
* Borders for win7 are below, win10 only needs something like this when
* drawing in the titlebar (-moz-windows-accent-color-in-titlebar). */
#main-window[sizemode="normal"] #navigator-toolbox:-moz-lwtheme {
border-top: 1px solid @glassShadowColor@;
}
@ -294,7 +295,8 @@
}
/* Artificially draw window borders that are covered by lwtheme, see bug 591930.
* We use a different border for win8, and this is not necessary on win10+ */
* We use a different border for win8, and this win10+ only needs this if
* drawing in the titlebar (-moz-windows-accent-color-in-titlebar). */
#main-window[sizemode="normal"] #navigator-toolbox:-moz-lwtheme {
border-top: 1px solid @glassActiveBorderColor@;
padding-top: 1px;