Page MenuHomePhabricator

Provide a richer "Revisions/Commits" UI in Maniphest
Closed, ResolvedPublic

Description

See PHI1508. Currently, commits and revisions on tasks are rendered in a slightly-enriched list view:

Screen Shot 2019-10-31 at 10.01.04 AM.png (159×800 px, 39 KB)

It might be nice to render these in a richer tabular format, somewhat similar to the task graph.

(The request in PHI1508 is narrowly about including repository information, specifically, but I suspect this UI can be broadly improved.)

Event Timeline

epriestley created this task.
epriestley renamed this task from Provide a richer "Changes" UI in Maniphest to Provide a richer "Revisions/Commits" UI in Maniphest.Nov 1 2019, 4:35 PM

Layout looks very neat but a strange too with that big space gap between the icon on each row and the actual text in this case:

image.png (362×780 px, 66 KB)

The empty space means "this is a normal commit with no special audit state", and the column collapses if no commits have issues (see this task for an example). That seems fairly reasonable to me?

I could put a grey "(No Audit Concerns)" in every row, but that doesn't seem very useful.

I could just use an icon (instead of a full tag with a text label) but I generally don't like that UI pattern very much and would like to move away from unlabeled / hover-to-understand icons in the UI where there isn't a very strong reason for an interface to work like that.

I could put a grey "(No Audit Concerns)" tag in every row, then remove them all if no row has a different status, I suppose.

Does this still strike you as weird after you've had some time to get used to it?

It wasn't a major UI issue to start with and yes, you get used to it. I would ignore.

Sounds good. It felt slightly odd to me at first, too, but I think I also got used to it.

I think Diffusion is going to get a similar treatment at some point, so maybe I'll revisit the UI then if there's some opportunity to adjust this in order to make the UIs more consistent or whatever.