From 545f24869ed180bea5804c3a2ccc8a449deb2f6a Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 4 Apr 2019 11:13:15 +0200
Subject: [PATCH 1/7] change panel ordering

---
 src/app/staticanalysis/staticAnalysisView.js  | 50 +++++++++----------
 .../styles/staticAnalysisView-styles.js       |  4 ++
 src/app/ui/styles/renderer-styles.js          |  2 +-
 3 files changed, 30 insertions(+), 26 deletions(-)

diff --git a/src/app/staticanalysis/staticAnalysisView.js b/src/app/staticanalysis/staticAnalysisView.js
index 699387b61a..b8f00c468f 100644
--- a/src/app/staticanalysis/staticAnalysisView.js
+++ b/src/app/staticanalysis/staticAnalysisView.js
@@ -43,32 +43,32 @@ staticAnalysisView.prototype.render = function () {
   var self = this
   var view = yo`
     <div class="${css.analysis}">
-      <div id="staticanalysismodules" class="list-group list-group-flush">
-        ${this.modulesView}
-      </div>
       <div class="${css.buttons}">
-        <div class="${css.buttonsInner}">
-          <button class="${css.buttonRun} btn btn-sm btn-primary" onclick="${function () { self.run() }}" >Run</button>
-          <label class="${css.label}" for="autorunstaticanalysis">
-            <input id="autorunstaticanalysis"
-              type="checkbox"
-              style="vertical-align:bottom"
-              checked="true"
-            >
-            Auto run
-          </label>
-          <label class="${css.label}" for="checkAllEntries">
-            <input id="checkAllEntries"
-              type="checkbox"
-              onclick="${function (event) { self.checkAll(event) }}"
-              style="vertical-align:bottom"
-              checked="true"
-            >
-            Check/Uncheck all
-          </label>
+          <div class="${css.buttonsInner}">
+            <button class="${css.buttonRun} btn btn-sm btn-primary" onclick="${function () { self.run() }}" >Run</button>
+            <label class="${css.label}" for="autorunstaticanalysis">
+              <input id="autorunstaticanalysis"
+                type="checkbox"
+                style="vertical-align:bottom"
+                checked="true"
+              >
+              Auto run
+            </label>
+            <label class="${css.label}" for="checkAllEntries">
+              <input id="checkAllEntries"
+                type="checkbox"
+                onclick="${function (event) { self.checkAll(event) }}"
+                style="vertical-align:bottom"
+                checked="true"
+              >
+              Check/Uncheck all
+            </label>
+          </div>
         </div>
-      </div>
-      <div class="${css.result}" "id='staticanalysisresult'></div>
+        <div id="staticanalysismodules" class="list-group list-group-flush ${css.container}">
+          ${this.modulesView}
+        </div>      
+        <div class="${css.result}" "id='staticanalysisresult'></div>
     </div>
   `
   if (!this.view) {
@@ -96,8 +96,8 @@ staticAnalysisView.prototype.run = function () {
   var selected = this.selectedModules()
   var warningContainer = $('#staticanalysisresult')
   warningContainer.empty()
+  var self = this
   if (this.lastCompilationResult) {
-    var self = this
     var warningCount = 0
     this.runner.run(this.lastCompilationResult, selected, function (results) {
       results.map(function (result, i) {
diff --git a/src/app/staticanalysis/styles/staticAnalysisView-styles.js b/src/app/staticanalysis/styles/staticAnalysisView-styles.js
index 4aa5f5450a..65b241178a 100644
--- a/src/app/staticanalysis/styles/staticAnalysisView-styles.js
+++ b/src/app/staticanalysis/styles/staticAnalysisView-styles.js
@@ -25,6 +25,10 @@ var css = csjs`
     display: flex;
     align-items: center;
   }
+  .container {
+    max-height: 500px;
+    overflow-y: auto;
+  }
 `
 
 module.exports = css
diff --git a/src/app/ui/styles/renderer-styles.js b/src/app/ui/styles/renderer-styles.js
index 710d7fb821..2a143a165a 100644
--- a/src/app/ui/styles/renderer-styles.js
+++ b/src/app/ui/styles/renderer-styles.js
@@ -49,7 +49,7 @@ var css = yo`<style>
 }
 
 .sol.staticAnalysisWarning {
-  background-color: var(--info);
+  background-color: var(--warning);
 }
 
 .sol.success {

From 4f9124dca7413b5bb20fab41c358b7986b83372d Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 4 Apr 2019 11:19:06 +0200
Subject: [PATCH 2/7] test tab reordering panel

---
 src/app/tabs/styles/test-tab-styles.js | 3 +++
 src/app/tabs/test-tab.js               | 6 +++---
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/app/tabs/styles/test-tab-styles.js b/src/app/tabs/styles/test-tab-styles.js
index 8eab2c9e0b..166e280beb 100644
--- a/src/app/tabs/styles/test-tab-styles.js
+++ b/src/app/tabs/styles/test-tab-styles.js
@@ -11,6 +11,9 @@ var css = csjs`
     display: flex;
     flex-direction: column;
     margin: 2%;
+    max-height: 500px;
+    overflow-y: auto;
+
   }
   .container {
     margin: 2%;
diff --git a/src/app/tabs/test-tab.js b/src/app/tabs/test-tab.js
index 78b140b833..e3093054b8 100644
--- a/src/app/tabs/test-tab.js
+++ b/src/app/tabs/test-tab.js
@@ -174,9 +174,8 @@ module.exports = class TestTab extends ApiFactory {
           How to test smart contracts guide in our documentation.
           <div class="${css.generateTestFile} btn btn-primary m-1" onclick="${this.testTabLogic.generateTestFile(this)}">Generate test file</div>
         </div>
-        <div class="${css.tests}">
-          ${this.testList}
-          <div class="${css.buttons} btn-group">
+        <div class="${css.tests}">          
+          <div class="${css.buttons}">
             <div class="${css.runButton} btn btn-primary m-1"  onclick="${this.runTests.bind(this)}">Run Tests</div>
             <label class="${css.label}" for="checkAllTests">
               <input id="checkAllTests"
@@ -187,6 +186,7 @@ module.exports = class TestTab extends ApiFactory {
               Check/Uncheck all
             </label>
           </div>
+          ${this.testList}
           ${this.testsOutput}
           ${this.testsSummary}
         </div>

From 196507328696aaa15e7e2db4f97c49ffde3ffba9 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 4 Apr 2019 11:35:22 +0200
Subject: [PATCH 3/7] minwidth for swap panel

---
 src/app.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app.js b/src/app.js
index b3a4a55070..70d7c3e947 100644
--- a/src/app.js
+++ b/src/app.js
@@ -170,7 +170,7 @@ class App extends ApiFactory {
 
   init () {
     var self = this
-    self._components.resizeFeature = new PanelsResize('#swap-panel', '#editor-container', { 'minWidth': 400, x: 450 })
+    self._components.resizeFeature = new PanelsResize('#swap-panel', '#editor-container', { 'minWidth': 300, x: 300 })
     run.apply(self)
   }
 

From 097767ee2044bcfa8ff030d8210b288e3586dd36 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 4 Apr 2019 11:35:41 +0200
Subject: [PATCH 4/7] minwidth for terminal

---
 src/app/panels/editor-panel.js | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/app/panels/editor-panel.js b/src/app/panels/editor-panel.js
index 7dd6652f67..91c4810042 100644
--- a/src/app/panels/editor-panel.js
+++ b/src/app/panels/editor-panel.js
@@ -79,10 +79,9 @@ class EditorPanel {
       self._view.mainPanel.style.display = 'none'
     })
     self.data = {
-      _FILE_SCROLL_DELTA: 200,
       _layout: {
         top: {
-          offset: self._deps.config.get('terminal-top-offset') || 500,
+          offset: self._deps.config.get('terminal-top-offset') || 150,
           show: true
         }
       }
@@ -104,7 +103,7 @@ class EditorPanel {
           var height = window.innerHeight
           var newpos = (event.pageY < limitUp) ? limitUp : event.pageY
           newpos = (newpos < height - limitDown) ? newpos : height - limitDown
-          return newpos
+          return height - newpos
         }
       })
 
@@ -138,11 +137,11 @@ class EditorPanel {
     var tmp = delta - limitDown
     delta = tmp > 0 ? tmp : 0
     if (direction === 'top') {
-      var height = containerHeight - delta
-      height = height < 0 ? 0 : height
-      self._view.editor.style.height = `${delta}px`
-      self._view.mainPanel.style.height = `${delta}px`
-      self._view.terminal.style.height = `${height}px` // - menu bar height
+      var mainPanelHeight = containerHeight - delta
+      mainPanelHeight = mainPanelHeight < 0 ? 0 : mainPanelHeight
+      self._view.editor.style.height = `${mainPanelHeight}px`
+      self._view.mainPanel.style.height = `${mainPanelHeight}px`
+      self._view.terminal.style.height = `${delta}px` // - menu bar height
       self._components.editor.resize((document.querySelector('#editorWrap') || {}).checked)
       self._components.terminal.scroll2bottom()
     }

From 1205d8637bfb0916b8b65e6ef2939add3d569acf Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 4 Apr 2019 11:42:20 +0200
Subject: [PATCH 5/7] focus on terminal at load

---
 src/app/panels/terminal.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/app/panels/terminal.js b/src/app/panels/terminal.js
index 741529a23f..cb056f35d8 100644
--- a/src/app/panels/terminal.js
+++ b/src/app/panels/terminal.js
@@ -104,12 +104,17 @@ class Terminal {
     if (opts.shell) self._shell = opts.shell
     register(self)
   }
+
+  focus () {
+    if (this._view.input) this._view.input.focus()
+  }
+
   render () {
     var self = this
     if (self._view.el) return self._view.el
     self._view.journal = yo`<div class=${css.journal}></div>`
     self._view.input = yo`
-      <span class=${css.input} spellcheck="false" contenteditable="true" onpaste=${paste} onkeydown=${change}></span>
+      <span class=${css.input} spellcheck="false" onload=${() => { this.focus() }} contenteditable="true" onpaste=${paste} onkeydown=${change}></span>
     `
     self._view.input.innerText = '\n'
     self._view.cli = yo`

From 02b80e98dd03fb2c2cdd6c2c1527e776dccd9143 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 4 Apr 2019 12:52:47 +0200
Subject: [PATCH 6/7] fix test & analysis view

---
 src/app/staticanalysis/staticAnalysisView.js               | 6 ++++--
 src/app/staticanalysis/styles/staticAnalysisView-styles.js | 4 +++-
 src/app/tabs/styles/test-tab-styles.js                     | 4 +++-
 src/app/tabs/test-tab.js                                   | 2 ++
 4 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/app/staticanalysis/staticAnalysisView.js b/src/app/staticanalysis/staticAnalysisView.js
index b8f00c468f..d8ce2c2fb5 100644
--- a/src/app/staticanalysis/staticAnalysisView.js
+++ b/src/app/staticanalysis/staticAnalysisView.js
@@ -67,8 +67,10 @@ staticAnalysisView.prototype.render = function () {
         </div>
         <div id="staticanalysismodules" class="list-group list-group-flush ${css.container}">
           ${this.modulesView}
-        </div>      
-        <div class="${css.result}" "id='staticanalysisresult'></div>
+        </div>
+        <hr>
+        <div><h6>Results:</h6></div>
+        <div class="${css.result}" id='staticanalysisresult'></div>
     </div>
   `
   if (!this.view) {
diff --git a/src/app/staticanalysis/styles/staticAnalysisView-styles.js b/src/app/staticanalysis/styles/staticAnalysisView-styles.js
index 65b241178a..4f157a8269 100644
--- a/src/app/staticanalysis/styles/staticAnalysisView-styles.js
+++ b/src/app/staticanalysis/styles/staticAnalysisView-styles.js
@@ -7,6 +7,8 @@ var css = csjs`
   }
   .result {
     margin-top: 1%;
+    max-height: 400px;
+    overflow-y: auto;
   }
   .buttons  {
     margin: 1rem 0;
@@ -26,7 +28,7 @@ var css = csjs`
     align-items: center;
   }
   .container {
-    max-height: 500px;
+    max-height: 400px;
     overflow-y: auto;
   }
 `
diff --git a/src/app/tabs/styles/test-tab-styles.js b/src/app/tabs/styles/test-tab-styles.js
index 166e280beb..ca05bd7eb6 100644
--- a/src/app/tabs/styles/test-tab-styles.js
+++ b/src/app/tabs/styles/test-tab-styles.js
@@ -11,13 +11,15 @@ var css = csjs`
     display: flex;
     flex-direction: column;
     margin: 2%;
-    max-height: 500px;
+    max-height: 300px;
     overflow-y: auto;
 
   }
   .container {
     margin: 2%;
     padding-bottom: 5%;
+    max-height: 300px;
+    overflow-y: auto;
   }
   .outputTitle {
     font-weight: bold;
diff --git a/src/app/tabs/test-tab.js b/src/app/tabs/test-tab.js
index e3093054b8..cdf1a30aa4 100644
--- a/src/app/tabs/test-tab.js
+++ b/src/app/tabs/test-tab.js
@@ -187,6 +187,8 @@ module.exports = class TestTab extends ApiFactory {
             </label>
           </div>
           ${this.testList}
+          <hr>
+          <div class="${css.buttons}" ><h6>Results:</h6></div>
           ${this.testsOutput}
           ${this.testsSummary}
         </div>

From 0cf4f2b7c8042260422c12c0cdbda0610cea2955 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 4 Apr 2019 17:38:38 +0200
Subject: [PATCH 7/7] change max-height

---
 src/app/staticanalysis/styles/staticAnalysisView-styles.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/app/staticanalysis/styles/staticAnalysisView-styles.js b/src/app/staticanalysis/styles/staticAnalysisView-styles.js
index 4f157a8269..1c38435dfc 100644
--- a/src/app/staticanalysis/styles/staticAnalysisView-styles.js
+++ b/src/app/staticanalysis/styles/staticAnalysisView-styles.js
@@ -7,7 +7,7 @@ var css = csjs`
   }
   .result {
     margin-top: 1%;
-    max-height: 400px;
+    max-height: 300px;
     overflow-y: auto;
   }
   .buttons  {
@@ -28,7 +28,7 @@ var css = csjs`
     align-items: center;
   }
   .container {
-    max-height: 400px;
+    max-height: 300px;
     overflow-y: auto;
   }
 `