Milo: Add an option to hide green steps

milo/appengine/frontend/templates/pages/build.html

 {{define "title"}}
   {{ with .Build.Summary -}}
     {{ if eq .Status.String "InfraFailure" }}
       Infra Failure
     {{ else if eq .Status.String "Exception" }}
       Exception
     {{ else if eq .Status.String "Failure" }}
       Failed
     {{ else if eq .Status.String "NotRun" }}
       Pending
@@ skipping 130 matching lines @@
 
       {{ if .Build.SourceStamp }}
         {{ if .Build.SourceStamp.Source }}
           <h2>Reason:</h2>
           <p>{{ .Source }}</p>
         {{ end }}
       {{ end }}
 
       {{ if or .Build.Components .Build.Summary.SubLink }}
         <h2>Steps and Logfiles:</h2>
-        <input type="checkbox" id="showHidden">
-        <label for="showHidden">Show hidden <span id="numHidden"></span></label>
-        <ol id="steps" class="hide">
+        Show:
+        <input type="radio" name="hider" id="showAll"><label for="showAll">All</label>
+        <input type="radio" name="hider" id="showStandard" checked><label for="showStandard">Standard</label>
+        <input type="radio" name="hider" id="showFail"><label for="showFail">Failure</label>

[nodir, 2017/05/18 02:41:04]

"Failures"? Since there may be multiple failures

[Ryan Tseng, 2017/05/18 22:52:49]

Actually i'll say "Interesting Only".  A bit verbose, but since "Running" is
included as interesting I think Failure(s) is the wrong word.

+        <ol id="steps" class="standard">
 
           {{ with .Build.Summary }}
             {{ if .SubLink }}
             <li class="verbosity-{{.Verbosity.String}}">
               <div class="status-{{.Status}} result">
                 <b>Steps</b>
                 {{ if .Duration -}}
                 <span style="float:right"
                       class="duration"
                       data-starttime="{{ .Started | formatTime }}"
@@ skipping 154 matching lines @@
 
       </li>
       {{ end }}
       </ol>
     </div>
     {{ end }}
   </div>
 </div>
 <script language="javascript">
 $(document).ready(function() {
-  var check = function(checked) {
+
+  var check = function(filter) {
     var things = $("#steps");

[nodir, 2017/05/18 02:41:04]

not used

[Ryan Tseng, 2017/05/18 22:52:48]

Done.

-    if (checked) {
-      $("#steps").removeClass("hide");
+    if (filter == "fail") {
+      $("#steps").removeClass("standard");
+      $("#steps").removeClass("all");
+      $("#steps").addClass("fail");
+    } else if (filter == "all") {
+      $("#steps").removeClass("standard");
+      $("#steps").addClass("all");
+      $("#steps").removeClass("fail");
     } else {
-      $("#steps").addClass("hide");
-    }
-    var numHidden = $(".verbosity-Hidden").length;
-    if (numHidden > 0) {
-      $("#numHidden").text("(" + numHidden + " hidden)");
-    } else {
-      $("#numHidden").text("");
+      // Default to standard.
+      $("#steps").removeClass("all");
+      $("#steps").addClass("standard");
+      $("#steps").removeClass("fail");
     }

[nodir, 2017/05/18 02:41:04]

I think this entire function can be implemented in one line

$("#steps").removeClass("all standard fail").addClass(filter)

[Ryan Tseng, 2017/05/18 22:52:48]

That works, that's awesome.

   };
 
-  check($("#showHidden").is(":checked"));
-  $("#showHidden").click(function(e) {
-    check($(this).is(":checked"));
+  check($("#showFail").is(":checked"), "standard");

[nodir, 2017/05/18 02:41:04]

i think you meant check("standard")

[nodir, 2017/05/18 03:36:16]

i recommend reviewing code yourself before sending for review. Then it is hard
to miss stuff like this.
I always do it for all my patchsets.

[Ryan Tseng, 2017/05/18 22:52:48]

Done.

[Ryan Tseng, 2017/05/18 22:52:48]

Acknowledged.

+
+  $("#showFail").click(function(e) {
+    check("fail");
+  });
+  $("#showStandard").click(function(e) {
+    check("standard");
+  });
+  $("#showAll").click(function(e) {
+    check("all");
   });
 });
 </script>
 {{end}}