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

Bug 1918618 - Cleanup content separator color handling. r=dao,desktop-theme-reviewers

Browse Source 
There's no need for a separate --lwt variable. In most themes
--chrome-content-separator-color gets set to the theme's
toolbar_bottom_separator anyway, so it wasn't particularly effective.

Differential Revision: https://phabricator.services.mozilla.com/D222089
This commit is contained in:
Emilio Cobos Álvarez 2024-09-13 10:19:55 +00:00
parent ba21ebd4f6
commit 9f33822580
3 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
browser/themes/ThemeVariableMap.sys.mjs
View File

@ -138,7 +138,7 @@ export const ThemeVariableMap = [
},
],
[
"--lwt-sidebar-border-color",
"--sidebar-border-color",
{
lwtProperty: "sidebar_border",
},

14
browser/themes/shared/browser-colors.css
View File

@ -10,7 +10,7 @@
--sidebar-background-color: -moz-sidebar;
--sidebar-text-color: -moz-sidebartext;
--sidebar-border-color: var(--lwt-sidebar-border-color, -moz-sidebarborder);
--sidebar-border-color: -moz-sidebarborder;
--button-color: currentColor;
--button-primary-bgcolor: AccentColor;
@ -41,12 +41,12 @@
}
--content-area-shadow: 0 2px 6px 0 light-dark(rgba(0,0,0,.2), rgba(0,0,0,.8));
--chrome-content-separator-color: var(--lwt-sidebar-border-color, light-dark(rgba(0,0,0,.3), rgba(255,255,255,.3)));
--chrome-content-separator-color: light-dark(rgba(0,0,0,.3), rgba(255,255,255,.3));
/* stylelint-disable-next-line media-query-no-invalid */
@media not (-moz-bool-pref: "sidebar.revamp") {
--chrome-content-separator-color: ThreeDShadow;
&[lwttheme] {
--chrome-content-separator-color: light-dark(rgba(0,0,0,.3), rgba(255,255,255,.3));
&:not([lwtheme]) {
--chrome-content-separator-color: ThreeDShadow;
}
}
@ -154,7 +154,9 @@
--urlbar-icon-fill-opacity: 1;
}
--chrome-content-separator-color: light-dark(rgb(204, 204, 204), hsl(240, 5%, 5%));
@media not (-moz-bool-pref: "sidebar.revamp") {
--chrome-content-separator-color: light-dark(rgb(204, 204, 204), hsl(240, 5%, 5%));
}
--link-color: light-dark(rgb(0, 97, 224), rgb(0, 221, 255));
--attention-dot-color: light-dark(#2ac3a2, #54ffbd);

2
browser/themes/shared/browser-shared.css
View File

@ -488,7 +488,7 @@ menupopup::part(drop-indicator) {
background-color: var(--toolbar-bgcolor);
color: var(--toolbar-color);
color-scheme: var(--toolbar-color-scheme);
border-top-color: var(--chrome-content-separator-color, ThreeDShadow);
border-top-color: var(--chrome-content-separator-color);
:root[lwtheme] & {
background-color: var(--lwt-accent-color);