HomePhabricator

Use a list view for DiffusionHistory

Tags
None
Referenced Files
F4980356: image.png
May 31 2017, 12:31 AM
Subscribers
None

Description

Use a list view for DiffusionHistory

Summary: This moves Diffusion History to use an easier to parse list view for commits and their (diff, audit, build) status. I left TableView around, which is used on a repositories home, and we can maybe add a "graph view" history back as another controller. Not sure what the real use is for that kind of feature though. I don't have Harbormaster set up locally so I could use another install to give this a run. I also expect to maybe not live with this UI as final, I like the UX, but the icons for indicating status don't really feel great to me, just OK.

Test Plan:
pull various repositories, check various history displays.

image.png (1×1 px, 392 KB)

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D18039

Details

Provenance
Chad Little <chad@phacility.com>Authored on May 28 2017, 12:55 AM
chadPushed on May 31 2017, 12:31 AM
Reviewer
epriestley
Differential Revision
D18039: Use a list view for DiffusionHistory
Parents
rP87c59c086741: Fix design atrocity
Branches
Unknown
Tags
Unknown
Tasks
T12824: Redesign Diffusion
Build Status
Buildable 17326
Build 23221: Run Core Tests