Backed out changeset f4a2f64e853f (bug 1573504) for debugger failure
authorCoroiu Cristina <ccoroiu@mozilla.com>
Wed, 21 Aug 2019 19:30:54 +0300
changeset 489244 3fbb7302bcb7b3b4e56b8f08cc32767a0acef057
parent 489243 f4a2f64e853ffe28f50ae5c72efe307556702b88
child 489245 ab9106acfa78902d08d8db70ba25f49fda8d00ae
push id93215
push userccoroiu@mozilla.com
push dateWed, 21 Aug 2019 16:32:33 +0000
treeherderautoland@3fbb7302bcb7 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1573504
milestone70.0a1
backs outf4a2f64e853ffe28f50ae5c72efe307556702b88
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
Backed out changeset f4a2f64e853f (bug 1573504) for debugger failure
devtools/client/debugger/src/components/Editor/InlinePreview.css
devtools/client/debugger/src/components/Editor/InlinePreview.js
devtools/client/debugger/src/components/variables.css
--- a/devtools/client/debugger/src/components/Editor/InlinePreview.css
+++ b/devtools/client/debugger/src/components/Editor/InlinePreview.css
@@ -3,25 +3,30 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 .inline-preview {
   position: relative;
   margin-top: calc(var(--theme-code-line-height) * -11px);
 }
 
 .inline-preview-outer {
-  padding: 1px 0;
-  background-color: var(--theme-inline-preview-background);
-  border: 1px solid var(--theme-inline-preview-border-color);
-  border-radius: 3px;
-  font-size: 10px;
+  padding: 2px;
+  border: 1px solid HSL(56.1, 87%, 65%);
+  opacity: 0.8;
+  background-color: HSL(56.1, 100%, 92.1%);
+  border-radius: 2px;
+  font-size: 9px;
+  color: HSL(56.1, 87%, 23%);
   margin-right: 5px;
 }
 
 .inline-preview-label {
-  padding: 1px 2px 1px 4px;
-  color: var(--theme-inline-preview-label-color);
-  background-color: var(--theme-inline-preview-label-background);
+  padding: 2px;
+  border-right: 1px solid HSL(56.1, 87%, 70%);
+  background-color: #fffab3;
+  border-radius: 2px 0 0 2px;
 }
 
 .inline-preview-value {
+  background-color: HSL(56.1, 100%, 96.1%);
   padding: 2px 6px;
+  border-radius: 0 2px 2px 0;
 }
--- a/devtools/client/debugger/src/components/Editor/InlinePreview.js
+++ b/devtools/client/debugger/src/components/Editor/InlinePreview.js
@@ -26,17 +26,17 @@ class InlinePreview extends PureComponen
 
   render() {
     const { value, variable } = this.props;
     return (
       <span
         className="inline-preview-outer"
         onClick={() => this.showInScopes(variable)}
       >
-        <span className="inline-preview-label">{variable}:</span>
+        <span className="inline-preview-label">{variable}</span>
         <span className="inline-preview-value">
           <Rep object={value} mode={MODE.SHORT} noGrip={true} />
         </span>
       </span>
     );
   }
 }
 
--- a/devtools/client/debugger/src/components/variables.css
+++ b/devtools/client/debugger/src/components/variables.css
@@ -12,23 +12,15 @@
   /* Remove once https://bugzilla.mozilla.org/show_bug.cgi?id=1520440 lands */
   --theme-code-line-height: calc(15 / 11);
 }
 
 :root.theme-light,
 :root .theme-light {
   --search-overlays-semitransparent: rgba(221, 225, 228, 0.66);
   --popup-shadow-color: #d0d0d0;
-  --theme-inline-preview-background: rgba(192, 105, 255, 0.05);
-  --theme-inline-preview-border-color: #EBD1FF;
-  --theme-inline-preview-label-color: #6300a6;
-  --theme-inline-preview-label-background: rgb(244, 230, 255);
 }
 
 :root.theme-dark,
 :root .theme-dark {
   --search-overlays-semitransparent: rgba(42, 46, 56, 0.66);
   --popup-shadow-color: #5c667b;
-  --theme-inline-preview-background: rgba(192, 105, 255, 0.05);
-  --theme-inline-preview-border-color: #47326C;
-  --theme-inline-preview-label-color: #DFCCFF;
-  --theme-inline-preview-label-background: #3F2E5F;
 }