Bug 1396037 - Part 2: Use the photon border and background colors in the breadcrumbs. r=pbro
authorGabriel Luong <gabriel.luong@gmail.com>
Mon, 04 Sep 2017 08:30:16 -0400
changeset 428342 67055f747a43e78808ea36fb118d7cb4fd68dbc9
parent 428341 e77976cfbab03a68bb27009549db4381a065c848
child 428343 a3037a3d59ebed30d297c662d35ab376d48e22a4
push id7761
push userjlund@mozilla.com
push dateFri, 15 Sep 2017 00:19:52 +0000
treeherdermozilla-beta@c38455951db4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspbro
bugs1396037
milestone57.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1396037 - Part 2: Use the photon border and background colors in the breadcrumbs. r=pbro
devtools/client/themes/inspector.css
devtools/client/themes/widgets.css
--- a/devtools/client/themes/inspector.css
+++ b/devtools/client/themes/inspector.css
@@ -6,24 +6,16 @@
 :root {
   --eyedropper-image: url(images/command-eyedropper.svg);
 }
 
 .theme-firebug {
   --eyedropper-image: url(images/firebug/command-eyedropper.svg);
 }
 
-:root.theme-light {
-  --breadcrumbs-border-color: #f3f3f3;
-}
-
-:root.theme-dark {
-  --breadcrumbs-border-color: #454d5d;
-}
-
 * {
   box-sizing: border-box;
 }
 
 /* Make sure to hide scroll bars for the parent window */
 window {
   overflow: hidden;
 }
@@ -119,20 +111,18 @@ window {
 #inspector-sidebar-toggle-box {
   line-height: initial;
 }
 
 #inspector-breadcrumbs-toolbar {
   padding: 0px;
   border-bottom-width: 0px;
   border-top-width: 1px;
-  border-top-color: var(--breadcrumbs-border-color);
-  /* Bug 1262668 - Use the same background as the body so the breadcrumbs toolbar doesn't
-     get mistaken as a splitter */
-  background-color: var(--theme-body-background);
+  border-top-color: var(--theme-splitter-color);
+  background-color: var(--theme-toolbar-background);
   display: block;
   position: relative;
 }
 
 #inspector-breadcrumbs-toolbar,
 #inspector-breadcrumbs-toolbar * {
   box-sizing: border-box;
 }
--- a/devtools/client/themes/widgets.css
+++ b/devtools/client/themes/widgets.css
@@ -228,17 +228,17 @@
   background-repeat: no-repeat;
   /* Given the 1/2 aspect ratio of the separator pseudo-element and the 45deg angle of
      the arrow shape, we need the arrow edges to be at this position from the start of
      the gradient line. */
   --position: 66.5%;
   /* The color of the thin line in the arrow-shaped separator between 2 unselected
      crumbs. There is no theme variable for this, this used to be an image. */
   --line-color: #ACACAC;
-  --background-color: var(--theme-body-background);
+  --background-color: var(--theme-toolbar-background);
 }
 
 #debugger-toolbar .breadcrumbs-widget-item::before {
   --background-color: var(--theme-toolbar-background);
 }
 
 .theme-dark .breadcrumbs-widget-item::before {
   --line-color: #6E6E6E;