Bug 65089 - Overhaul garden-o-matic frontend to integrate revision history and failures
Summary: Overhaul garden-o-matic frontend to integrate revision history and failures
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on:
Blocks: 64188
  Show dependency tree / graph
 
Reported: 2011-07-24 17:23 PDT by Adam Barth
Modified: 2011-07-25 10:51 PDT (History)
5 users (show)

See Also:

Attachments
Patch (36.58 KB, patch)
2011-07-24 17:26 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
Patch (38.27 KB, patch)
2011-07-24 17:27 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
screenshot of the new UI (220.26 KB, image/png)
2011-07-24 17:29 PDT, Adam Barth 		no flags Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2011-07-24 17:23:50 PDT 
Overhaul garden-o-matic frontend to integrate revision history and failures
Comment 1 Adam Barth 2011-07-24 17:26:55 PDT 
Created attachment 101840 [details]
Patch
Comment 2 Adam Barth 2011-07-24 17:27:41 PDT 
Created attachment 101841 [details]
Patch
Comment 3 Adam Barth 2011-07-24 17:29:27 PDT 
Created attachment 101842 [details]
screenshot of the new UI

Obviously there's a lot of work still to be done on the UX, but this is at least a good foundation upon which to build.  This patch is not particularly well tested.  I'll probably do another iteration with better tests.

Note: The four white columns on the left are the Windows builders, which do not properly report their SVN revision.  We need to fix that on the bot!
Comment 4 Dimitri Glazkov (Google) 2011-07-25 08:45:03 PDT 
Comment on attachment 101841 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=101841&action=review

> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.css:184
> +    text-decoration: none;

Adding multiples of these can be simplified with an extra selector .test-name::after { text-decoration: none; }
Comment 5 Adam Barth 2011-07-25 10:35:23 PDT 
> Adding multiples of these can be simplified with an extra selector .test-name::after { text-decoration: none; }

Oh, good idea.
Comment 6 WebKit Review Bot 2011-07-25 10:51:21 PDT 
Comment on attachment 101841 [details]
Patch

Clearing flags on attachment: 101841

Committed r91677: <http://trac.webkit.org/changeset/91677>
Comment 7 WebKit Review Bot 2011-07-25 10:51:27 PDT 
All reviewed patches have been landed.  Closing bug.