Page MenuHomePhabricator

Convert commits to use unified table of contents
ClosedPublic

Authored by epriestley on Aug 15 2015, 6:41 PM.
Tags
None
Referenced Files
F14352653: D13910.id33573.diff
Thu, Dec 19, 2:28 PM
Unknown Object (File)
Sat, Dec 14, 5:18 PM
Unknown Object (File)
Thu, Dec 12, 6:11 AM
Unknown Object (File)
Thu, Dec 12, 12:26 AM
Unknown Object (File)
Tue, Nov 26, 7:47 PM
Unknown Object (File)
Tue, Nov 26, 1:29 PM
Unknown Object (File)
Sat, Nov 23, 12:56 PM
Unknown Object (File)
Thu, Nov 21, 9:46 PM
Subscribers
None

Details

Summary

Fixes T2183. We now use the same rendering element in both places.

Intentional changes:

  • Package highlighting is out, coming back to both apps in next diff.
  • removed redundant-feeling "Change" link. The information is now shown with a character ("M", "V", etc.) and the page is a click away under "History". Clicking the path also jumps you to substantially similar content. (We could restore it fairly easily, I just think it's probably the least useful thing in the table right now.)
Test Plan

Viewed a bunch of commits in Diffusion.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Convert commits to use unified table of contents.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 15 2015, 7:19 PM
This revision was automatically updated to reflect the committed changes.