From 1dc53ecff26e69677295ba4c8a2c5387e594b81e Mon Sep 17 00:00:00 2001
From: LianaHus <liana@ethdev.com>
Date: Tue, 12 May 2020 12:44:05 +0200
Subject: [PATCH] changed the opacity of highlight color added opacity on
 highlighitcode_fullLine removed anused css class

---
 src/app/editor/sourceHighlighter.js       | 4 ++--
 src/app/ui/styles/renderer-styles.js      | 6 ------
 test-browser/tests/staticAnalysis.test.js | 4 ++--
 3 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/src/app/editor/sourceHighlighter.js b/src/app/editor/sourceHighlighter.js
index 73f8181239..55920b3900 100644
--- a/src/app/editor/sourceHighlighter.js
+++ b/src/app/editor/sourceHighlighter.js
@@ -47,7 +47,7 @@ class SourceHighlighter {
         .highlightcode {
           position:absolute;
           z-index:20;
-          opacity: 0.2;
+          opacity: 0.3;
           background-color: ${style || 'var(--info)'};
         }
         .highlightcode_fullLine {
@@ -61,7 +61,7 @@ class SourceHighlighter {
         }
         `
 
-      this.statementMarker = this._deps.editor.addMarker(lineColumnPos, this.source, css.highlightcode.className + ' ' + css.customBackgroundColor.className + ' ' + `highlightLine${lineColumnPos.start.line}`)
+      this.statementMarker = this._deps.editor.addMarker(lineColumnPos, this.source, css.highlightcode.className +  ' ' + css.customBackgroundColor.className + ' ' + `highlightLine${lineColumnPos.start.line}`)
       this._deps.editor.scrollToLine(lineColumnPos.start.line, true, true, function () {})
       this.position = lineColumnPos
       if (lineColumnPos.start.line === lineColumnPos.end.line) {
diff --git a/src/app/ui/styles/renderer-styles.js b/src/app/ui/styles/renderer-styles.js
index 05627ecd25..7dc047c440 100644
--- a/src/app/ui/styles/renderer-styles.js
+++ b/src/app/ui/styles/renderer-styles.js
@@ -3,7 +3,6 @@ var yo = require('yo-yo')
 var css = yo`<style>
 .sol.success,
 .sol.error,
-.sol.staticAnalysisWarning,
 .sol.warning {
     white-space: pre-line;
     word-wrap: break-word;
@@ -17,7 +16,6 @@ var css = yo`<style>
 
 .sol.success pre,
 .sol.error pre,
-.sol.staticAnalysisWarning pre,
 .sol.warning pre {
     white-space: pre-line;
     overflow-y: hidden;
@@ -30,7 +28,6 @@ var css = yo`<style>
 }
 
 .sol.success .close,
-.sol.staticAnalysisWarning .close,
 .sol.error .close,
 .sol.warning .close {
     white-space: pre-line;
@@ -48,9 +45,6 @@ var css = yo`<style>
 .sol.warning {
 }
 
-.sol.staticAnalysisWarning {
-}
-
 .sol.success {
   /* background-color:  // styles.rightPanel.message_Success_BackgroundColor; */
 }</style>`
diff --git a/test-browser/tests/staticAnalysis.test.js b/test-browser/tests/staticAnalysis.test.js
index d97d8f1973..4ee4183475 100644
--- a/test-browser/tests/staticAnalysis.test.js
+++ b/test-browser/tests/staticAnalysis.test.js
@@ -38,11 +38,11 @@ function runTests (browser) {
     .testContracts('Untitled.sol', sources[0]['browser/Untitled.sol'], ['TooMuchGas', 'test1', 'test2'])
     .clickLaunchIcon('solidityStaticAnalysis')
     .click('#staticanalysisView button')
-    .waitForElementPresent('#staticanalysisresult .staticAnalysisWarning', 2000, true, function () {
+    .waitForElementPresent('#staticanalysisresult .warning', 2000, true, function () {
       listSelectorContains(['Use of tx.origin',
         'Fallback function of contract TooMuchGas requires too much gas',
         'TooMuchGas.() : Variables have very similar names test and test1.'],
-        '#staticanalysisresult .staticAnalysisWarning',
+        '#staticanalysisresult .warning',
         browser, function () {
           browser.end()
         }