Simplify Diffusion main view
ClosedPublic
Actions

Authored by epriestley on May 12 2014, 10:51 PM.

Details

Reviewers

btrahan
chad

Commits

Restricted Diffusion Commit
rP77b4c3145a8b: Simplify Diffusion main view

Summary

Currently, Diffusion has very complex views. After three years I'm not really used to them and rarely use many of these options.

Simplify the browse and history views:

Put the browse view on top.
Move dates to the right.
Remove "History" and "Edit" links from the browse view. You can access these actions by clicking the file/path.
Remove "Browse" link from the history view. You can access this action by clicking the commit.
Remove "Change Type", which is essentially never useful, from the history view.
Add some tweaks for mobile.

Test Plan

Screen_Shot_2014-05-12_at_3.51.04_PM.png (1×1 px, 346 KB)

Diff Detail

Repository

rP Phabricator

Branch

browse1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 412
Build 412: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 21588.May 12 2014, 10:51 PM

epriestley retitled this revision from to Simplify Diffusion main view.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: btrahan, chad.

epriestley added a subscriber: zeeg.

Herald added a subscriber: epriestley. · View Herald TranscriptMay 12 2014, 10:51 PM

Harbormaster completed remote builds in B412: Diff 21588.May 12 2014, 10:52 PM

Do we need two columns for date / time. I see that in many tables, but not sure the usefulness.

This revision is now accepted and ready to land.May 12 2014, 10:54 PM

I think it makes it easier to read -- Mail.app does it so it must be right? We could collapse them if you prefer, I don't have strong feelings about it.