Differential D20799

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

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

Tags

None

Referenced Files

	F18752506: D20799.diff
	Sat, Oct 4, 3:29 PM

	F18734031: D20799.diff
	Tue, Sep 30, 10:47 PM

	F18649725: D20799.id49593.diff
	Sep 20 2025, 10:05 PM

	F18570625: D20799.id49594.diff
	Sep 10 2025, 4:00 AM

	F18570622: D20799.id49593.diff
	Sep 10 2025, 4:00 AM

	F18567379: D20799.id.diff
	Sep 9 2025, 3:44 PM

	F18462659: D20799.diff
	Sep 1 2025, 10:52 PM

	F17987642: D20799.id.diff
	Aug 1 2025, 10:32 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T8808: Policy menu should show actual project icon next to project
Commits: rPd2e1c4163a3b: When a project has a custom icon, use that icon to label the project policy in…

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

Screen Shot 2019-09-09 at 1.33.48 PM.png (219×322 px, 20 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Sep 9 2019, 8:35 PM

Harbormaster completed remote builds in B23418: Diff 49593.Sep 9 2019, 8:37 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.

Closed by commit rPd2e1c4163a3b: When a project has a custom icon, use that icon to label the project policy in… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPd2e1c4163a3b: When a project has a custom icon, use that icon to label the project policy in….

Revision Contents
Changeset List

Path

Size

src/

applications/

policy/

storage/

PhabricatorPolicy.php

6 lines

Diff 49594

View Options

src/applications/policy/storage/PhabricatorPolicy.php

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