Page MenuHomePhabricator

Roll out more FontIcons
ClosedPublic

Authored by chad on Jan 26 2015, 2:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 7:03 AM
Unknown Object (File)
Thu, Apr 25, 1:25 AM
Unknown Object (File)
Tue, Apr 16, 12:37 AM
Unknown Object (File)
Thu, Apr 11, 3:48 AM
Unknown Object (File)
Wed, Apr 10, 8:10 PM
Unknown Object (File)
Apr 6 2024, 7:59 AM
Unknown Object (File)
Mar 28 2024, 4:36 AM
Unknown Object (File)
Mar 23 2024, 9:33 PM
Subscribers
Tokens
"Love" token, awarded by johnny-bit.

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP96edc9d2bc8f: Roll out more FontIcons
Summary

Sidenav launcher, search typeahead results, apps launcher

Test Plan

Used each of these items

pasted_file (735×223 px, 101 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Roll out more FontIcons.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added inline comments.
src/applications/diffusion/application/PhabricatorDiffusionApplication.php
22

Went back to code here. The silhouette is much easier to pick out than the file code icon, and it's very easy to spot in the sidebar now.

  • Remove remaining use of apps-sprite-large
epriestley edited edge metadata.

Cool. In practice, I think these work a lot better than I'd expected they might.

This revision is now accepted and ready to land.Jan 26 2015, 3:18 PM
This revision was automatically updated to reflect the committed changes.