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>

[nodir, 2017/05/22 16:36:38]

nit: maybe move label to the next line so it is consistent with other checkboxes

[Ryan Tseng, 2017/05/22 17:29:46]

Done.

+        <input type="radio" name="hider" id="showStandard" checked>
+          <label for="showStandard">Standard</label>
+        <input type="radio" name="hider" id="showInteresting">
+          <label for="showInteresting">Interesting Only</label>
+        <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 things = $("#steps");
-    if (checked) {
-      $("#steps").removeClass("hide");
-    } else {
-      $("#steps").addClass("hide");
+
+  var check = function(filter) {

[nodir, 2017/05/22 16:36:38]

i think the parameter here is a css class name. filter may be easily confused
with jquery filter (e.g. .classSelector or #id)

[Ryan Tseng, 2017/05/22 17:29:46]

Done.

+    $("#steps").removeClass("all standard interesting").addClass(filter)
+  }
+
+  $("#showInteresting").click(function(e) {
+    check("interesting");
+  });
+  $("#showStandard").click(function(e) {
+    check("standard");
+  });
+  $("#showAll").click(function(e) {
+    check("all");
+  });
+
+  $("input[name='hider']").each(function(){
+    if ($(this).prop("checked") == true) {

[nodir, 2017/05/22 16:36:38]

why not `if ($(this).prop("checked")) {` ?

[Ryan Tseng, 2017/05/22 17:29:46]

I saw it written the other way in another example snippet.  I'm not sure why
they did it that way either.  I chalked it up to javascript conventions.

+      $(this).click();
     }
-    var numHidden = $(".verbosity-Hidden").length;
-    if (numHidden > 0) {
-      $("#numHidden").text("(" + numHidden + " hidden)");
-    } else {
-      $("#numHidden").text("");
-    }
-  };
-
-  check($("#showHidden").is(":checked"));
-  $("#showHidden").click(function(e) {
-    check($(this).is(":checked"));
   });
 });
 </script>
 {{end}}