Page MenuHomePhabricator

Convert rest of SPRITE_STATUS to FontAwesome
ClosedPublic

Authored by chad on May 18 2014, 5:53 AM.
Tags
None
Referenced Files
F13259829: D9179.diff
Sun, May 26, 11:09 PM
F13244541: D9179.diff
Thu, May 23, 5:08 AM
F13235986: D9179.id21820.largetrue.diff
Tue, May 21, 7:26 AM
Unknown Object (File)
Tue, May 7, 5:31 AM
Unknown Object (File)
Mon, May 6, 11:47 PM
Unknown Object (File)
Mon, May 6, 11:37 AM
Unknown Object (File)
Fri, May 3, 3:25 AM
Unknown Object (File)
Mon, Apr 29, 2:42 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).