Bug 1444301 - Update close icon to use Photon icon; r?jryans draft
authorBrian Birtles <birtles@gmail.com>
Thu, 05 Apr 2018 10:13:22 +0900
changeset 777595 4690cf3ad8d9709e2f1425e14edf21547347b715
parent 777594 12f15d8d2a1c34e1d76aee91ea223b96ea8e66c2
child 777596 71e0f05c6be88ae41cd4673390deed9996d84bf7
push id105256
push userbmo:bbirtles@mozilla.com
push dateThu, 05 Apr 2018 01:56:46 +0000
reviewersjryans
bugs1444301
milestone61.0a1
Bug 1444301 - Update close icon to use Photon icon; r?jryans This is the icon taken from https://design.firefox.com/icons/viewer/#close MozReview-Commit-ID: B3BdbJf1W1f
devtools/client/themes/images/close.svg
devtools/client/themes/toolbox.css
--- a/devtools/client/themes/images/close.svg
+++ b/devtools/client/themes/images/close.svg
@@ -1,6 +1,6 @@
 <!-- This Source Code Form is subject to the terms of the Mozilla Public
    - 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/. -->
-<svg width="16" height="16" viewBox="0 0 16 16" xmlns="http://www.w3.org/2000/svg" fill="context-fill #0b0b0b">
-  <path d="M8.707 8l4.23 4.23a.5.5 0 1 1-.707.707L8 8.707l-4.23 4.23a.5.5 0 1 1-.707-.707L7.293 8l-4.23-4.23a.5.5 0 1 1 .707-.707L8 7.293l4.23-4.23a.5.5 0 0 1 .707.707L8.707 8z" fill-rule="evenodd"/>
+<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" viewBox="0 0 16 16">
+  <path fill="context-fill" d="M9.061 8l3.47-3.47a.75.75 0 0 0-1.061-1.06L8 6.939 4.53 3.47a.75.75 0 1 0-1.06 1.06L6.939 8 3.47 11.47a.75.75 0 1 0 1.06 1.06L8 9.061l3.47 3.47a.75.75 0 0 0 1.06-1.061z"></path>
 </svg>
--- a/devtools/client/themes/toolbox.css
+++ b/devtools/client/themes/toolbox.css
@@ -191,16 +191,17 @@
 /* Save space in Firebug theme */
 .theme-firebug #toolbox-controls button {
   margin-inline-start: 0 !important;
   min-width: 12px;
   margin: 0 1px;
 }
 
 #toolbox-close::before {
+  fill: var(--theme-toolbar-photon-icon-color);
   background-image: var(--close-button-image);
 }
 
 #toolbox-menu-button::before {
   fill: var(--theme-toolbar-photon-icon-color);
   background-image: var(--more-button-image);
 }