Merge pull request #564 from ethereum/scrollbars

Remove reduntant scrollbars

assets/css/browser-solidity.css

@@ -134,6 +134,7 @@ body {
     bottom: 0;
     overflow: auto;
     box-sizing: border-box;
+    overflow: hidden;
 }
 
 #header {
@@ -150,7 +151,6 @@ body {
 
 #header #menu {
     border-bottom: 3px solid #F4F6FF;
-    overflow: hidden;
     height: 2em;
     font-size: 1em;
     padding-top: 0.5em;

src/app/analysis-tab.js

@@ -9,6 +9,7 @@ var css = csjs`
   .analysisTabView {
     padding: 2%;
     margin-top: 1em;
+    padding-bottom: 3em;
   }
   .infoBox extends ${styles.infoTextBox} {
     margin-bottom: 1em;

src/universal-dapp.js

@@ -21,14 +21,17 @@ var styleGuide = require('./app/style-guide')
 var styles = styleGuide()
 
 var css = csjs`
+  html {
+    overflow: hidden;
+  }
   .options {
-      float: left;
-      padding: 0.7em 0.3em;
-      font-size: 0.9em;
-      cursor: pointer;
-      background-color: transparent;
-      margin-right: 0.5em;
-      font-size: 1em;
+    float: left;
+    padding: 0.7em 0.3em;
+    font-size: 0.9em;
+    cursor: pointer;
+    background-color: transparent;
+    margin-right: 0.5em;
+    font-size: 1em;
   }
   .title extends ${styles.titleBox} {
     cursor: pointer;

test-browser/tests/compiling.js

@@ -33,7 +33,11 @@ function runTests (browser) {
       .click('.instance .call[title="f"]')
       .waitForElementPresent('.output .returned')
       .assert.containsText('.output .returned', '0x0000000000000000000000000000000000000000000000000000000000000008')
-      .assert.containsText('.output .decoded li', 'uint256: 8')
-      .end()
+      .execute(function () {
+        document.querySelector('.output .decoded li').scrollIntoView()
+      }, [], function () {
+        browser.assert.containsText('.output .decoded li', 'uint256: 8')
+        .end()
+      })
   })
 }