984599 - Display test jobs using a table

Status: RESOLVED WONTFIX

(Tree Management :: Treeherder, enhancement, P5)

Description

[Jesse Ruderman]

Attachment: mockup (Firefox only)

Using a table-column for each job type would make it easier to see if:
* mochitest-1 is busted on all platforms
* mochitest-5 has not yet finished on any platforms

The attached mockup also switches from letters to colored boxes. This compressed view leaves more room for what I think are the important things:
* commit comments
* a list of test failures (bug 983017)

Comment 1

[Phil Ringnalda (:philor)]

There's at least one previous bug, which I think said something about "add padding", which also missed the fact that it should be showing enormous blank areas, since as a single horrible example of the whole problem of reality not conforming to the idea of neat ordered columns, we have 15 hunks of mochitests on b2g debug.

Comment 2

[Jesse Ruderman]

Bug 626255 suggested adding padding. Thanks.

I'm not too worried about adding "enormous blank areas". TBPL as it exists now has lots those, along with enormous *non-blank* areas that convey a tiny bit of useful information in a format that's difficult to skim. Furthermore, one table for desktop and one table for mobile would almost eliminate them.

Comment 3

[(no longer active)]

I like the idea here.  I have one major issue with this, which is, this hides the name of the suites by default, but that should be easy to address by adding a header for each column.

The other is the accessibility story, I don't want this change to make TBPL inaccessible.  Alexander, can we use some ARIA magic here?

About comment 1, I actually think that if we add smaller squares for each of these letters, we might be able to fit things in the current width.

Comment 4

[(no longer active)]

CCing other a11y gurus as well.

Comment 5

[alexander :surkov (:asurkov)]

HTML tables (table headings) are accessible by default in general. What do you need ARIA for?

Comment 6

[Trevor Saunders (:tbsaunde)]

> The other is the accessibility story, I don't want this change to make TBPL
> inaccessible.  Alexander, can we use some ARIA magic here?

looking at the attachment this actually seems somewhat better when you move to a table cell instead of just getting the job type you get job type + platform because the I guess the platform is a row header or something.  On the other hand it doesn't fix the issue that you need to select a job to tell what state its in (although that's atleast sort of a bug between gecko and orca)

Comment 7

[Ed Morley [:emorley]]

I'm extremely keen for us to experiment with alternative UIs in treeherder (the replacement for TBPL) - once it's up and running I'd really like to try something like this out.

Comment 8

[David Burns :automatedtester]

We have moved to a manifest view, experimental on treeherder at the moment. Also with the way we do chunking it is not possible without bloating the UI.