Page MenuHomePhabricator

New icons for Projects
ClosedPublic

Authored by chad on Jun 27 2017, 10:37 AM.

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.