Page MenuHomePhabricator

Remove white app icons
ClosedPublic

Authored by chad on May 12 2014, 6:47 PM.
Tags
None
Referenced Files
F13817623: D9078.diff
Fri, Sep 20, 1:33 AM
F13814734: D9078.id21583.largetrue.diff
Thu, Sep 19, 6:51 AM
F13814733: D9078.id21583.diff
Thu, Sep 19, 6:50 AM
F13814732: D9078.id.largetrue.diff
Thu, Sep 19, 6:49 AM
F13814728: D9078.id.diff
Thu, Sep 19, 6:49 AM
F13814727: D9078.diff
Thu, Sep 19, 6:49 AM
F13814510: D9078.largetrue.diff
Thu, Sep 19, 5:41 AM
Unknown Object (File)
Mon, Sep 16, 5:26 AM

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP315ac6aab7cc: Remove white app icons
Summary

Remove white app icons, no longer in use as far as grep/memory serve. These were for list hover states.

Test Plan

Rebuild sprites, celerity. Grep for appIcon use (only feed). Verify all action lists are driven by FontAwesome.

Diff Detail

Repository
rP Phabricator
Branch
white-apps
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 408
Build 408: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad retitled this revision from to Remove white app icons.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 12 2014, 6:50 PM

i will update with quick create menu before landing.

chad edited edge metadata.
  • Swap app-icons in quick create menu to fonts
chad updated this revision to Diff 21583.

Closed by commit rP315ac6aab7cc (authored by @chad).