Page MenuHomePhabricator

Convert rest of SPRITE_STATUS to FontAwesome
ClosedPublic

Authored by chad on May 18 2014, 5:53 AM.
Tags
None
Referenced Files
F14092490: D9179.diff
Mon, Nov 25, 6:23 AM
Unknown Object (File)
Tue, Nov 19, 9:41 AM
Unknown Object (File)
Thu, Nov 14, 10:50 PM
Unknown Object (File)
Wed, Nov 13, 10:44 PM
Unknown Object (File)
Wed, Nov 13, 10:42 PM
Unknown Object (File)
Mon, Nov 11, 1:14 PM
Unknown Object (File)
Wed, Nov 6, 10:54 PM
Unknown Object (File)
Tue, Oct 29, 10:56 PM
Subscribers

Details

Summary

Updates policy, headers, typeaheads to FA over policy icons

Need advice - can't seem to place where icons come from on Typeahead? Wrong icons and wrong colors.... it is late

Test Plan
  • grepped for SPRITE_STATUS
  • grepped for sprite-status
  • grepped for setStatus for headers
  • grepped individual icons names

Browsed numerous places, checked new dropdowns, see pudgy people.

Diff Detail

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

Event Timeline

chad retitled this revision from to Convert rest of SPRITE_STATUS to FontAwesome.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

I think the inlines fix the typeahead. This should get cleaner after T4420.

src/applications/typeahead/controller/PhabricatorTypeaheadCommonDatasourceController.php
201

Make this fa-user bluegrey.

291

fa-briefcase bluegrey

This revision is now accepted and ready to land.May 18 2014, 12:11 PM
chad updated this revision to Diff 21820.

Closed by commit rP3a81f8c68d29 (authored by @chad).