Page MenuHomePhabricator
Differential D14096

Clean up browse/history links in Diffusion
ClosedPublic
Actions

Authored by epriestley on Sep 11 2015, 1:23 AM.
Tags
None
Referenced Files
F18788207: D14096.id34071.diff
Wed, Oct 15, 3:31 AM
F18736266: D14096.diff
Wed, Oct 1, 5:17 AM
F18712738: D14096.id.diff
Mon, Sep 29, 5:31 AM
F18680104: D14096.id.diff
Fri, Sep 26, 2:14 AM
F18659118: D14096.id34069.diff
Tue, Sep 23, 9:59 AM
F18506464: D14096.id.diff
Sep 5 2025, 1:30 AM
F18499066: D14096.diff
Sep 4 2025, 7:21 PM
F18351493: D14096.diff
Aug 26 2025, 10:06 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9126: diffusion history page should give browse links for each committed version of the file
Commits
Restricted Diffusion Commit
rP4e181a5611fb: 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)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 34069.Sep 11 2015, 1:23 AM
epriestley retitled this revision from to Clean up browse/history links in Diffusion.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T9126: diffusion history page should give browse links for each committed version of the file.
Herald edited edge metadata. · View Herald TranscriptSep 11 2015, 1:23 AM
chad accepted this revision.Sep 11 2015, 2:26 AM
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 11 2015, 2:26 AM
Closed by commit rP4e181a5611fb: Clean up browse/history links in Diffusion (authored by epriestley, committed by epriestley). · Explain WhySep 11 2015, 2:28 AM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T9126: diffusion history page should give browse links for each committed version of the file.Sep 11 2015, 3:23 AM

Revision Contents
Changeset List

PathSize
resources/
celerity/
62 lines
src/
applications/
differential/
constants/
15 lines
diffusion/
controller/
20 lines
13 lines
7 lines
view/
35 lines
31 lines
21 lines
115 lines
infrastructure/
diff/
view/
2 lines
webroot/
rsrc/
css/
aphront/
6 lines
application/
diffusion/
33 lines
image/
icon/
fatcow/

Diff 34071

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/differential/constants/DifferentialChangeType.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionHistoryController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionRepositoryController.php

Loading...
View Options

src/applications/diffusion/view/DiffusionBrowseTableView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionEmptyResultView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionHistoryTableView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionView.php

Loading...
View Options

src/infrastructure/diff/view/PHUIDiffTableOfContentsListView.php

Loading...
View Options

webroot/rsrc/css/aphront/table-view.css

Loading...
View Options

webroot/rsrc/css/application/diffusion/diffusion-icons.css

Loading...
View Options

webroot/rsrc/image/icon/fatcow/folder.png

Loading...
View Options

webroot/rsrc/image/icon/fatcow/folder_go.png

Loading...
View Options

webroot/rsrc/image/icon/fatcow/page_white_link.png

Loading...
View Options

webroot/rsrc/image/icon/fatcow/page_white_text.png

Loading...
Phabricator · Built by Phacility