These are old project image choices, remove and only go with FontAwesome related images.
Details
- Reviewers
epriestley - Commits
- Restricted Diffusion Commit
rPf7646b40aace: Remove defunt project image choices
Project -> Edit Picture -> Save
Diff Detail
- Repository
- rP Phabricator
- Branch
- project-icons (branched from master)
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 10188 Build 12373: Run Core Tests Build 12372: arc lint + arc unit
Event Timeline
There's no real reason not to keep these around as extra options, unless you don't want to or worry that there will be way too many options once we fully slice FA?
I'd imagine possibly using this composer UI for Phame (and maybe even Users?) in the future, so having even more variety seems sort of nice. Although it could get overwhelming if you have 2000 options to pick from, but we could do grouping or something eventually.
(That said, I don't think we use the actual sprite or have any plans to do so, nor the 1x images, since we always compose PNGs from the sources.)
I roughly plan to land a handful of fa equivalent. I just didn't want to overload this diff. "All" fa is a bit overkill unless its a typeahead.
Oh, I suppose most of these actually do have near-duplicates in FA and the rest I'm not terribly attached to (like "8ball").
We were using the sprite sheet in the file composer, so I re-wrote that as fonts instead.
Ah, sure. We can swap that for composed builtins too if we do want to add non-FA images eventually.