Bug 1374812 - Update sidebar header button hover states. r=dao
MozReview-Commit-ID: HDg3gwI5Pfa
--- a/browser/themes/shared/sidebar.inc.css
+++ b/browser/themes/shared/sidebar.inc.css
@@ -64,37 +64,37 @@
width: 12px;
height: 12px;
}
#sidebar-switcher-target {
-moz-appearance: none;
color: inherit;
margin-inline-end: 4px;
- border-radius: 4px;
+ border-radius: var(--toolbarbutton-border-radius);
border: 1px solid transparent;
padding: 2px 4px;
}
#sidebar-switcher-target:hover {
- background: hsla(240, 5%, 5%, 0.05);
-}
-
-#sidebar-switcher-target:hover {
- border-color: rgba(0, 0, 0, 0.2);
+ background: var(--toolbarbutton-hover-background);
}
#sidebar-switcher-target:hover:active,
#sidebar-switcher-target.active {
- background: hsla(240, 5%, 5%, 0.1);
+ background: var(--toolbarbutton-active-background);
}
-#sidebar-switcher-target:hover:active,
-#sidebar-switcher-target.active {
- border-color: rgba(0, 0, 0, 0.25);
+/* Ensure we do not lose contrast between lightweight and OS theme colours */
+#sidebar-switcher-target:hover:-moz-lwtheme {
+ background: hsla(240, 5%, 5%, 0.1);
+}
+#sidebar-switcher-target:hover:active:-moz-lwtheme,
+#sidebar-switcher-target.active:-moz-lwtheme {
+ background: hsla(240, 5%, 5%, 0.15);
}
#sidebarMenu-popup .subviewbutton {
min-width: 190px;
}
#sidebar-extensions:empty + toolbarseparator {
display: none;