Page MenuHomePhabricator

Remove 1x AppIcons, use FontIcons instead
ClosedPublic

Authored by chad on Jan 25 2015, 9:53 PM.
Tags
None
Referenced Files
F13148574: D11496.diff
Sat, May 4, 2:36 AM
Unknown Object (File)
Thu, Apr 25, 2:51 AM
Unknown Object (File)
Sun, Apr 14, 9:47 PM
Unknown Object (File)
Sun, Apr 14, 9:42 PM
Unknown Object (File)
Sun, Apr 14, 8:57 PM
Unknown Object (File)
Thu, Apr 11, 10:16 AM
Unknown Object (File)
Tue, Apr 9, 11:34 PM
Unknown Object (File)
Mon, Apr 8, 9:58 PM
Tokens
"Love" token, awarded by johnny-bit.

Details

Summary

Removes the 1x application icons, and uses the fonticons instead. Feed was only known location.

Test Plan

feed, dashboards, grep for use

pasted_file (519×223 px, 61 KB)

Diff Detail

Repository
rP Phabricator
Branch
feed-icons
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4040
Build 4053: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad retitled this revision from to Remove 1x AppIcons, use FontIcons instead.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/transactions/feed/PhabricatorApplicationTransactionFeedStory.php
77

(Extra space after -> here.)

This revision is now accepted and ready to land.Jan 25 2015, 9:55 PM
chad edited edge metadata.
  • fix spacing
  • Remove old icon calls
This revision was automatically updated to reflect the committed changes.