Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(49)

Unified Diff: milo/appengine/frontend/expectations/buildbot.build-Debug_page-_CrWinGoma_30608.html

Issue 2886353002: Milo: Add an option to hide green steps (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: milo/appengine/frontend/expectations/buildbot.build-Debug_page-_CrWinGoma_30608.html
diff --git a/milo/appengine/frontend/expectations/buildbot.build-Debug_page-_CrWinGoma_30608.html b/milo/appengine/frontend/expectations/buildbot.build-Debug_page-_CrWinGoma_30608.html
index 655f667848a410d594d454a19860321682ce4471..57a338b70a94823e3e7dcc42c5f4b80ac45db563 100644
--- a/milo/appengine/frontend/expectations/buildbot.build-Debug_page-_CrWinGoma_30608.html
+++ b/milo/appengine/frontend/expectations/buildbot.build-Debug_page-_CrWinGoma_30608.html
@@ -107,9 +107,11 @@
<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>
+ <ol id="steps" class="standard">
@@ -1336,24 +1338,35 @@ Cr-Commit-Position: refs/heads/master@{#398661}
</div>
<script language="javascript">
$(document).ready(function() {
- var check = function(checked) {
+
+ var check = function(filter) {
var things = $("#steps");
- if (checked) {
- $("#steps").removeClass("hide");
- } else {
- $("#steps").addClass("hide");
- }
- var numHidden = $(".verbosity-Hidden").length;
- if (numHidden > 0) {
- $("#numHidden").text("(" + numHidden + " hidden)");
+ 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 {
- $("#numHidden").text("");
+
+ $("#steps").removeClass("all");
+ $("#steps").addClass("standard");
+ $("#steps").removeClass("fail");
}
};
- check($("#showHidden").is(":checked"));
- $("#showHidden").click(function(e) {
- check($(this).is(":checked"));
+ check($("#showFail").is(":checked"), "standard");
+
+ $("#showFail").click(function(e) {
+ check("fail");
+ });
+ $("#showStandard").click(function(e) {
+ check("standard");
+ });
+ $("#showAll").click(function(e) {
+ check("all");
});
});
</script>

Powered by Google App Engine
This is Rietveld 408576698