From a0cb59fbc3a868a290220dd7f325e2c5963668c5 Mon Sep 17 00:00:00 2001 From: Dmitriy Vyukov Date: Fri, 17 Oct 2014 11:32:43 +0400 Subject: [PATCH] dashboard: highlight positive changes Initially both positive and negative changes had solid background (green and red). Then we removed background of positive changes due to: https://code.google.com/p/go/issues/detail?id=8624 However, Ian noted: "I did intuitively understand that + was an increase and - was a decrease. I did not notice how the numbers were segregated between the third and fourth columns. Now that you point it out, it is obvious. It would be more obvious if the numbers in the fourth column were highlighted in green." Give positive changes green *border* (not background), so they are still visually distinguishable from negative even in grayscale. This is especially beneficial for perf detail view, because currently it looks like everything is either negative or neutral in that view (only red and black). LGTM=adg R=adg CC=golang-codereviews, iant, rsc https://golang.org/cl/159970043 --- dashboard/app/static/style.css | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dashboard/app/static/style.css b/dashboard/app/static/style.css index b99892aaae..ddf21296c7 100644 --- a/dashboard/app/static/style.css +++ b/dashboard/app/static/style.css @@ -197,7 +197,7 @@ table thead tr { text-overflow: ellipsis; } -.good { text-decoration: none; color: #000000; } +.good { text-decoration: none; color: #000000; border: 2px solid #00E700} .bad { text-decoration: none; text-shadow: 1px 1px 0 #000000; color: #FFFFFF; background: #E70000;} .noise { text-decoration: none; color: #888; } .fail { color: #C00; }