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 420221 a31957c189915081e63a5ed4393494802805b910
parent 420220 df864b81b63ae359c9ffe6527ecec745a234e7f9
child 420222 8569fdbf5a12c3ac0479217e62a84144bfdb9ad3
push id34068
push usernerli@mozilla.com
push dateTue, 29 May 2018 21:40:19 +0000
treeherdermozilla-central@5852258260e1 [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() ?