Page MenuHomePhabricator

When a project has a custom icon, use that icon to label the project policy in the policy dropown
ClosedPublic

Authored by epriestley on Sep 9 2019, 8:35 PM.

Details

Summary

Fixes T8808. Currently, all project use the default ("Briefcase") project icon when they appear in a policy dropdown.

Since project policies are separated out into a "Members of Projects" section of the dropdown anyway, there is no reason not to use the actual project icon, which is often more clear.

Test Plan

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Sep 9 2019, 8:35 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 9 2019, 8:37 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.