color fixes (#5314)

ericawright · jasonLaster · commit 0edd9cf3a13b · 2018-02-06T17:57:59.000-05:00
diff --git a/src/components/PrimaryPanes/Sources.css b/src/components/PrimaryPanes/Sources.css
@@ -51,10 +51,6 @@
   transform: rotate(0deg);
 }
 
-.theme-dark .sources-list .tree .node:not(.focused) img.arrow {
-  background: var(--theme-content-color3);
-}
-
 .theme-dark .source-list .tree .node.focused {
   background-color: var(--theme-tab-toolbar-background);
 }
diff --git a/src/components/SecondaryPanes/Expressions.css b/src/components/SecondaryPanes/Expressions.css
@@ -39,8 +39,7 @@
 .input-expression::placeholder {
   text-align: center;
   font-style: italic;
-  color: var(--theme-comment-alt);
-  opacity: 1;
+  color: var(--theme-comment);
 }
 
 .input-expression:focus {
diff --git a/src/components/SecondaryPanes/SecondaryPanes.css b/src/components/SecondaryPanes/SecondaryPanes.css
@@ -43,10 +43,6 @@
   cursor: default;
 }
 
-.theme-dark .secondary-panes .accordion .arrow svg {
-  fill: var(--theme-content-color3);
-}
-
 .secondary-panes .breakpoints-buttons {
   display: flex;
 }
diff --git a/src/components/WelcomeBox.css b/src/components/WelcomeBox.css
@@ -55,6 +55,7 @@
   text-align: left;
   float: left;
   margin-left: 25px;
+  color: var(--theme-comment);
 }
 
 html .welcomebox .toggle-button-end.collapsed {
diff --git a/src/components/shared/Accordion.css b/src/components/shared/Accordion.css
@@ -4,10 +4,12 @@
 
 :root {
   --accordion-header-background: var(--theme-toolbar-background);
+  --disclosure-arrow: #b2b2b2;
 }
 
 :root.theme-dark {
   --accordion-header-background: #222225;
+  --disclosure-arrow: #7f7f81;
 }
 
 .accordion {
@@ -78,3 +80,7 @@
 .accordion .header-buttons button::-moz-focus-inner {
   border: none;
 }
+
+.accordion .arrow svg {
+  fill: var(--disclosure-arrow);
+}
diff --git a/src/components/shared/Button/Close.css b/src/components/shared/Button/Close.css
@@ -16,7 +16,7 @@
 .close-btn .close {
   mask: url(/images/close.svg) no-repeat;
   mask-size: 100%;
-  background-color: var(--theme-comment-alt);
+  background-color: var(--theme-comment);
   width: 8px;
   height: 8px;
   transition: all 0.15s ease-in-out;
diff --git a/src/components/shared/SearchInput.css b/src/components/shared/SearchInput.css
@@ -57,7 +57,7 @@
 
 .search-field .magnifying-glass path,
 .search-field .magnifying-glass ellipse {
-  stroke: var(--theme-splitter-color);
+  stroke: var(--theme-comment);
 }
 
 .search-field input::placeholder {
diff --git a/src/components/shared/Svg.css b/src/components/shared/Svg.css
@@ -79,7 +79,7 @@ img.arrow {
   width: 9px;
   height: 9px;
   padding-top: 9px;
-  background: var(--theme-splitter-color);
+  background: var(--disclosure-arrow);
   mask-size: 100%;
   display: inline-block;
   margin-bottom: 1px;

Original file line number	Diff line number	Diff line change
`@@ -51,10 +51,6 @@`
`51`	`51`	`transform: rotate(0deg);`
`52`	`52`	`}`
`53`	`53`
`54`		`-.theme-dark .sources-list .tree .node:not(.focused) img.arrow {`
`55`		`- background: var(--theme-content-color3);`
`56`		`-}`
`57`		`-`
`58`	`54`	`.theme-dark .source-list .tree .node.focused {`
`59`	`55`	`background-color: var(--theme-tab-toolbar-background);`
`60`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,8 +39,7 @@`
`39`	`39`	`.input-expression::placeholder {`
`40`	`40`	`text-align: center;`
`41`	`41`	`font-style: italic;`
`42`		`- color: var(--theme-comment-alt);`
`43`		`- opacity: 1;`
	`42`	`+ color: var(--theme-comment);`
`44`	`43`	`}`
`45`	`44`
`46`	`45`	`.input-expression:focus {`
Original file line number	Diff line number	Diff line change
`@@ -43,10 +43,6 @@`
`43`	`43`	`cursor: default;`
`44`	`44`	`}`
`45`	`45`
`46`		`-.theme-dark .secondary-panes .accordion .arrow svg {`
`47`		`- fill: var(--theme-content-color3);`
`48`		`-}`
`49`		`-`
`50`	`46`	`.secondary-panes .breakpoints-buttons {`
`51`	`47`	`display: flex;`
`52`	`48`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,7 @@`
`55`	`55`	`text-align: left;`
`56`	`56`	`float: left;`
`57`	`57`	`margin-left: 25px;`
	`58`	`+ color: var(--theme-comment);`
`58`	`59`	`}`
`59`	`60`
`60`	`61`	`html .welcomebox .toggle-button-end.collapsed {`
Original file line number	Diff line number	Diff line change
`@@ -4,10 +4,12 @@`
`4`	`4`
`5`	`5`	`:root {`
`6`	`6`	`--accordion-header-background: var(--theme-toolbar-background);`
	`7`	`+ --disclosure-arrow: #b2b2b2;`
`7`	`8`	`}`
`8`	`9`
`9`	`10`	`:root.theme-dark {`
`10`	`11`	`--accordion-header-background: #222225;`
	`12`	`+ --disclosure-arrow: #7f7f81;`
`11`	`13`	`}`
`12`	`14`
`13`	`15`	`.accordion {`
`@@ -78,3 +80,7 @@`
`78`	`80`	`.accordion .header-buttons button::-moz-focus-inner {`
`79`	`81`	`border: none;`
`80`	`82`	`}`
	`83`	`+`
	`84`	`+.accordion .arrow svg {`
	`85`	`+ fill: var(--disclosure-arrow);`
	`86`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@`
`57`	`57`
`58`	`58`	`.search-field .magnifying-glass path,`
`59`	`59`	`.search-field .magnifying-glass ellipse {`
`60`		`- stroke: var(--theme-splitter-color);`
	`60`	`+ stroke: var(--theme-comment);`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`.search-field input::placeholder {`