Page MenuHomePhabricator

New icons for Projects
ClosedPublic

Authored by chad on Jun 27 2017, 10:37 AM.
Tags
None
Referenced Files
F15489871: D18162.id43694.diff
Fri, Apr 11, 1:01 PM
F15488598: D18162.id.diff
Thu, Apr 10, 11:52 PM
F15482678: D18162.diff
Wed, Apr 9, 7:11 AM
F15452684: D18162.id43694.diff
Sat, Mar 29, 6:42 AM
F15429672: D18162.id43691.diff
Mon, Mar 24, 3:33 AM
F15391519: D18162.id43691.diff
Mar 15 2025, 10:20 AM
F15391246: D18162.id43691.diff
Mar 15 2025, 9:01 AM
F15311732: D18162.diff
Mar 6 2025, 10:31 AM
Subscribers

Details

Summary

Updates the builtin images, leaves the old choose... icons for now. I'd like to automate this based on icon when creating a project.

Test Plan

Visit edit picture page, pick a few. Purge cache, see new default image.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

patiently awaits color commentary

epriestley added inline comments.
resources/builtin/project.png
1

The latch on this bag appears to be closed but one of our core values is OPENNESS.

resources/builtin/projects/v3/bug.png
1

Are these legs attached to the ABDOMEN instead of the THORAX? Ridiculous!

resources/builtin/projects/v3/calendar.png
1

This one is exactly accurate.

resources/builtin/projects/v3/people.png
1

I'm pretty sure this is two elves in tunics and that is great.

This revision is now accepted and ready to land.Jun 27 2017, 12:50 PM
This revision was automatically updated to reflect the committed changes.

Maybe one real piece of feedback is to consider a neutral or more varied palette of skin tones for the elves, they're pretty whitebread.

good idea. tempted to put a klingon on the right rear too.