Bug 1460209 - Use theme colors for links in application panel;r=ladybenko
authorJulian Descottes <jdescottes@mozilla.com>
Fri, 25 May 2018 11:43:59 +0200
changeset 420238 a31957c189915081e63a5ed4393494802805b910
parent 420237 df864b81b63ae359c9ffe6527ecec745a234e7f9
child 420239 8569fdbf5a12c3ac0479217e62a84144bfdb9ad3
push id64564
push userjdescottes@mozilla.com
push dateTue, 29 May 2018 14:10:23 +0000
treeherderautoland@a31957c18991 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersladybenko
bugs1460209
milestone62.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 1460209 - Use theme colors for links in application panel;r=ladybenko MozReview-Commit-ID: 69kfQ4137WA
devtools/client/application/src/components/App.css
devtools/client/application/src/components/Worker.js
--- a/devtools/client/application/src/components/App.css
+++ b/devtools/client/application/src/components/App.css
@@ -18,24 +18,24 @@ h1 {
 
 a {
   margin: 0 10px;
 }
 
 a,
 a:hover,
 a:visited {
-  color: var(--blue-60) !important;
+  color: var(--theme-highlight-blue) !important;
   cursor: pointer;
 }
 
 a.disabled,
 a.disabled:hover,
 a.disabled:visited {
-  color: var(--grey-30) !important;
+  opacity: 0.5 !important;
   cursor: default;
 }
 
 /*
  * The current layout of the application panel is
  *
  *  +---------------------------------------------+
  *  | (header) "Service workers"                  |
--- a/devtools/client/application/src/components/Worker.js
+++ b/devtools/client/application/src/components/Worker.js
@@ -119,17 +119,18 @@ class Worker extends Component {
       button({
         onClick: this.unregister,
         className: "devtools-button worker__unregister-button js-unregister-button",
         "data-standalone": true
       },
         Strings.GetStringFromName("unregister"))
       : null;
 
-    const debugLinkDisabled = this.isRunning() ? "" : "worker__debug-link--disabled";
+    const debugLinkDisabled = this.isRunning() ? "" : "disabled";
+
     const debugLink = a({
       onClick: this.isRunning() ? this.debug : null,
       title: this.isRunning() ? null : "Only running service workers can be debugged",
       className: `${debugLinkDisabled} worker__debug-link js-debug-link`
     },
       Strings.GetStringFromName("debug"));
 
     const startLink = !this.isRunning() ?