HomePhabricator

Clean up browse/history links in Diffusion

Tags
None
Referenced Files
F788422: Screen Shot 2015-09-10 at 6.17.52 PM.png
Sep 11 2015, 2:28 AM
F788420: Screen Shot 2015-09-10 at 6.17.21 PM.png
Sep 11 2015, 2:28 AM
F788419: Screen Shot 2015-09-10 at 6.18.05 PM.png
Sep 11 2015, 2:28 AM
F788421: Screen Shot 2015-09-10 at 6.17.44 PM.png
Sep 11 2015, 2:28 AM
Subscribers
None

Description

Clean up browse/history links in Diffusion

Summary:
Fixes T9126. In particular:

  • Add "Browse" links to all history views.
  • Use icons to show "Browse" and "History" links, instead of text.
  • Use FontAwesome.
  • Generally standardize handling of these elements.

This might need a little design attention, but I think it's an improvement overall.

Test Plan:

  • Viewed repository history.
  • Viewed branch history.
  • Viewed file history.
  • Viewed table of contents on a commit.
  • Viewed merged changes on a merge commit.
  • Viewed a directory containing an external.
  • Viewed a deleted file.

Screen Shot 2015-09-10 at 6.18.05 PM.png (1×1 px, 194 KB)

Screen Shot 2015-09-10 at 6.17.21 PM.png (1×1 px, 291 KB)

Screen Shot 2015-09-10 at 6.17.44 PM.png (1×1 px, 254 KB)

Screen Shot 2015-09-10 at 6.17.52 PM.png (1×1 px, 321 KB)

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T9126

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