Bug 1460209 - Use theme colors for links in application panel;r=ladybenko draft
authorJulian Descottes <jdescottes@mozilla.com>
Fri, 25 May 2018 11:43:59 +0200
changeset 800927 b7d77c1d3893255627c0ba1e9e9b51bc6b88545c
parent 799734 4d0695668607a7eb80d3815140d9ee09dae60dca
child 800928 f54639855b83a32fe43add241a6b7ab6688b6eb3
push id111519
push userjdescottes@mozilla.com
push dateTue, 29 May 2018 14:08:01 +0000
reviewersladybenko
bugs1460209
milestone62.0a1
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
@@ -118,17 +118,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() ?