Closed
Bug 1050078
Opened 9 years ago
Closed 9 years ago
the information about the build should have a verbal description of its status
Categories
(Tree Management :: Treeherder, defect, P2)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dbaron, Assigned: jfrench)
References
()
Details
Attachments
(2 files)
When you click on a build/test in treeherder, it shows a box in the lower left (as it does in TBPL). However, this box doesn't, anywhere, contain the word "pending" or "running" or similar to describe the status. This should be present for two reasons: (1) it avoids having people make distinctions based on color alone, which is extremely poor for accessibility (2) it provides a better way for people to learn the UI by exploring, which is often a better way to learn than a help page In TBPL's build data box, it's one of the things shown most prominently, in the top row.
Updated•9 years ago
|
Blocks: treeherder-sheriff-transition
Priority: -- → P2
Assignee | ||
Comment 1•9 years ago
|
||
Makes sense to me also from a UI standpoint. The UI is already there in Treeherder at the top of that lower left panel and displaying correctly for: Result: success Result: busted Result: testfailed Result: retry And we'd just want to add: Result: pending Result: running ...for suitable jobs, which presently just show as: Result: unknown
Assignee | ||
Comment 2•9 years ago
|
||
And that job.result property is specified for the panel here https://github.com/mozilla/treeherder-ui/blob/master/webapp/app/plugins/pluginpanel.html#L131
Assignee | ||
Comment 3•9 years ago
|
||
So (apologies.. newbie here..) in the job artifact, it appears we have job.result and job.state. They both seem correct in a pending and running job (ie. result is 'unknown'), and both legitimate pieces of information which inform each other. What do you think about including both in that current line for all jobs? I screen grabbed several results, attached. We could tune the padding on that line for more white space if desired.
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Comment 4•9 years ago
|
||
I spoke with jeads on IRC; he's in agreement so will open a PR based on the above concept shortly.
Assignee | ||
Comment 5•9 years ago
|
||
I've opened a PR per above comments. Please see the PR for review and status.
Comment 6•9 years ago
|
||
This looks good :-) The only thing I'd change (and happy to do this as a followup if needs be) is to display something other than "unknown" for the result of pending/running. eg "-", "N/A", "TBC" etc. I guess this also applies to the other fields that we don't yet know, eg machine "unknown" -> "unallocated" / "not allocated".
Assignee | ||
Comment 7•9 years ago
|
||
Ok, sounds good, let's do that as a follow up bug. 'unknown' comes straight from the job artifact, so I assume we would want it modified there, rather than converting it in the platform.
Comment 8•9 years ago
|
||
Yeah :-)
Assignee | ||
Comment 9•9 years ago
|
||
Above now entered as bug 1052501.
Assignee | ||
Comment 10•9 years ago
|
||
Merged and fixed in https://github.com/mozilla/treeherder-ui/commit/2df99999e5d9ccdc64537ca79deaa7756ed42fd7#diff-d41d8cd98f00b204e9800998ecf8427e I will marked resolved once it's pushed to dev/stage, and I've had a chance to check it live.
Assignee | ||
Comment 11•9 years ago
|
||
This has been pushed to stage and everything appears to be working as desired, so marking fixed. The new dark grey background color in the details pane for 'running' jobs could perhaps still be a tiny bit darker - but then the black text inside begins to lose readability. If we still want it a bit darker we can always try that later.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•