Page MenuHomePhabricator

Show packages in table of contents views in Diffusion and Differential
ClosedPublic

Authored by epriestley on Aug 17 2015, 4:44 PM.
Tags
None
Referenced Files
F15465462: D13923.diff
Wed, Apr 2, 7:03 PM
F15448491: D13923.id33599.diff
Fri, Mar 28, 5:13 AM
F15425559: D13923.id33599.diff
Sun, Mar 23, 4:55 AM
F15413474: D13923.diff
Wed, Mar 19, 7:02 PM
F15399292: D13923.id33599.diff
Mon, Mar 17, 3:49 AM
F15388915: D13923.id33599.diff
Sat, Mar 15, 4:38 AM
F15382630: D13923.diff
Fri, Mar 14, 1:30 PM
F15358007: D13923.id33599.diff
Tue, Mar 11, 7:08 AM
Subscribers
None

Details

Summary

Fixes T8004.

  • For paths which are part of a package, show the package.
  • Highlight paths which are part of a package you (the viewer) have authority over.
Test Plan

Screen Shot 2015-08-17 at 9.39.21 AM.png (184×1 px, 20 KB)

  • Viewed owned and unowned chagnes in Diffusion and Differential.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Show packages in table of contents views in Diffusion and Differential.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.

el testo

This revision is now accepted and ready to land.Aug 17 2015, 5:12 PM

I tend to think these might be better with just an icon or something, but El Testo Packageo seemed to fit OK.

This revision was automatically updated to reflect the committed changes.