Page MenuHomePhabricator

Add icons to Settings
ClosedPublic

Authored by epriestley on Jan 21 2019, 6:32 PM.
Tags
None
Referenced Files
F18784619: D20006.id47764.diff
Mon, Oct 13, 1:37 PM
F18630591: D20006.id47807.diff
Sep 16 2025, 12:07 PM
F18630176: D20006.diff
Sep 16 2025, 10:56 AM
F18606938: D20006.id47764.diff
Sep 13 2025, 10:43 PM
F18600000: D20006.diff
Sep 13 2025, 8:46 AM
F18597541: D20006.diff
Sep 13 2025, 3:10 AM
F18569284: D20006.id47807.diff
Sep 10 2025, 1:11 AM
F18569283: D20006.id47770.diff
Sep 10 2025, 1:11 AM
Subscribers
None

Details

Summary

Depends on D20005. I love icons.

Test Plan

Screen Shot 2019-01-21 at 10.31.38 AM.png (1×1 px, 291 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/settings/panel/PhabricatorTokensSettingsPanel.php
13–15

This is the only one I'm a little on the fence about, but I guess you're trying to make these unique and you don't want to re-use fa-key? Maybe fa-clock would work better? This is 100% USDA-certified bike shedding, but I feel like fa-hourglass connotes a thing that the user is waiting on to finish so they can get back to work, not a thing that is already good to go.

This revision is now accepted and ready to land.Jan 21 2019, 6:41 PM

Yeah, v1 of this had nine copies of fa-key and that didn't feel great. I'll fish around for something else, maybe fa-ticket.

This revision was automatically updated to reflect the committed changes.