manual UI also looking good

web-platform-tests · Mar 21, 2014 · 2bb35b1 · 2bb35b1
1 parent 3f37c7a
commit 2bb35b1
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 26 deletions.
diff --git a/tools/runner/index.html b/tools/runner/index.html
@@ -117,24 +117,26 @@ <h4>Progress</h4>
 
     <div class="results">
       <div id="manualUI">
-        <h4>Manual Testing</h4>
-        <p>
-          The current test requires manual result marking.
-        </p>
-        <div class="reftestUI">
-          <p>
-            Test and ref should compare <strong class="refType"></strong>
-          </p>
-          <p>
-            <button class="btn btn-info test">Show Test</button>
-            <button class="btn btn-info ref">Show Reference</button>
-          </p>
+        <div class='panel panel-primary'>
+          <div class='panel-heading'>
+            <h4 class='panel-title'>Manual Testing</h4>
+          </div>
+          <div class="panel-body reftestUI">
+            <p>
+              The current test requires manual result marking.
+              Test and ref should compare <strong class="refType text-primary"></strong>
+            </p>
+            <p>
+              <button class="btn btn-info test">Show Test</button>
+              <button class="btn btn-info ref">Show Reference</button>
+            </p>
+          </div>
+          <div class="panel-footer">
+            The test has:
+            <button class="btn btn-success pass">Passed</button>
+            <button class="btn btn-danger fail">Failed</button>
+          </div>
         </div>
-        <p>
-          Test has:
-          <button class="btn btn-success pass">Passed</button>
-          <button class="btn btn-danger fail">Failed</button>
-        </p>
       </div>
 
       <hr>

diff --git a/tools/runner/runner.css b/tools/runner/runner.css
@@ -1,7 +1,7 @@
-@charset "UTF-8";
+
 html { margin: 0 8px; }
 body { margin: 0; }
-html.done { border: 2px solid limegreen; margin: 3px; padding: 3px; }
+html.done { border: 2px solid #32cd32; margin: 3px; padding: 3px; }
 
 html:not(.done) { height: 100%; }
 html:not(.done) body { height: 100%; }
@@ -27,13 +27,19 @@ html:not(.done) section { border-top: none; }
 html.done section + section { border-top: none; }
 
 #manualUI {
-    position:fixed;
-    bottom:0;
-    left:0;
-    right:0;
+    position:   fixed;
+    top:    100px;
+    left:   0;
+    right:  0;
     display: block;
-    background-color:yellow;
-    padding:0 5em 0 5em;
+    padding:    40px;
+    background: rgba(255, 200, 0, 0.5);
+}
+
+#manualUI .panel {
+    max-width:  800px;
+    margin: auto;
+    box-shadow: 2px 2px 10px #666;
 }
 
 body > p { text-align: center; }

diff --git a/tools/runner/runner.js b/tools/runner/runner.js
@@ -506,7 +506,7 @@ Runner.prototype = {
     },
 
     done: function() {
-        this.test_window.close();
+        if (this.test_window) this.test_window.close();
         this.done_callbacks.forEach(function(callback) {
             callback();
         });