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
Unknown Object (File)
Sat, Jan 18, 12:46 AM
Unknown Object (File)
Mon, Jan 6, 10:13 PM
Unknown Object (File)
Sat, Dec 28, 4:45 PM
Unknown Object (File)
Wed, Dec 25, 1:08 AM
Unknown Object (File)
Sat, Dec 21, 10:48 AM
Unknown Object (File)
Fri, Dec 20, 11:08 AM
Unknown Object (File)
Dec 19 2024, 2:28 PM
Unknown Object (File)
Dec 14 2024, 5:18 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.