corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1915805 - Make sure revamped sidebar is opaque. r=desktop-theme-reviewers,sidebar-reviewers,dao,nsharpley

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D223620
This commit is contained in:
Emilio Cobos Álvarez
2024-09-25 19:19:04 +00:00
parent 8e551546f5
commit ddeb6eef91
2 changed files with 18 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
browser/components/sidebar/sidebar-main.css
View File

@@ -10,8 +10,6 @@
min-width: 50px; min-width: 50px;
padding-block: var(--space-small); padding-block: var(--space-small);
padding-inline: 0; padding-inline: 0;
background-color: var(--toolbar-bgcolor);
color: var(--toolbar-color);
/* stylelint-disable-next-line media-query-no-invalid */ /* stylelint-disable-next-line media-query-no-invalid */
@media (-moz-bool-pref: "sidebar.verticalTabs") { @media (-moz-bool-pref: "sidebar.verticalTabs") {

29
browser/themes/shared/sidebar.css
View File

@@ -2,12 +2,6 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this * 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/. */ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
#sidebar-main {
background-color: var(--toolbar-bgcolor);
color: var(--toolbar-color);
text-shadow: none;
}
#sidebar-box { #sidebar-box {
background-color: var(--sidebar-background-color); background-color: var(--sidebar-background-color);
color: var(--sidebar-text-color); color: var(--sidebar-text-color);
@@ -26,11 +20,6 @@
/* stylelint-disable-next-line media-query-no-invalid */ /* stylelint-disable-next-line media-query-no-invalid */
@media (-moz-bool-pref: "sidebar.revamp") { @media (-moz-bool-pref: "sidebar.revamp") {
min-width: 200px; min-width: 200px;
/* Note that with sidebar.revamp we apply the background to the #sidebar
* browser. We still need the #sidebar-box background to match the toolbar
* tho */
background-color: var(--toolbar-bgcolor);
color: var(--toolbar-color);
padding-block-end: var(--space-small); padding-block-end: var(--space-small);
padding-inline-end: var(--space-small); padding-inline-end: var(--space-small);
position: relative; position: relative;
@@ -50,6 +39,24 @@
} }
} }
/* Note that with sidebar.revamp we apply the --sidebar-background-color to the
* #sidebar browser. We still need the #sidebar-box background to match the
* toolbar tho */
/* stylelint-disable-next-line media-query-no-invalid */
@media (-moz-bool-pref: "sidebar.revamp") {
#sidebar-main,
#sidebar-box {
background-color: var(--toolbar-bgcolor);
color: var(--toolbar-color);
text-shadow: none;
:root[lwtheme] & {
background-color: var(--lwt-accent-color);
background-image: linear-gradient(var(--toolbar-bgcolor), var(--toolbar-bgcolor));
}
}
}
#sidebar-header { #sidebar-header {
font-size: 1.333em; font-size: 1.333em;
padding: 8px; padding: 8px;