Page MenuHomePhabricator

Roll out more FontIcons
ClosedPublic

Authored by chad on Jan 26 2015, 2:36 AM.
Tags
None
Referenced Files
F14092464: D11499.diff
Mon, Nov 25, 6:16 AM
Unknown Object (File)
Thu, Nov 21, 7:24 AM
Unknown Object (File)
Sun, Nov 17, 8:10 AM
Unknown Object (File)
Oct 24 2024, 2:17 PM
Unknown Object (File)
Oct 22 2024, 7:13 PM
Unknown Object (File)
Oct 22 2024, 11:17 AM
Unknown Object (File)
Oct 22 2024, 11:10 AM
Unknown Object (File)
Oct 22 2024, 11:09 AM
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.