Page MenuHomePhabricator

New Tokens
ClosedPublic

Authored by chad on Jul 1 2016, 4:10 AM.
Tags
None
Referenced Files
F19796868: D16211.diff
Fri, Feb 27, 5:33 PM
F19736750: D16211.id39000.diff
Sun, Feb 15, 4:13 AM
F19275437: D16211.diff
Dec 23 2025, 12:35 AM
F19018154: D16211.diff
Nov 23 2025, 10:15 AM
F18851589: D16211.id38996.diff
Oct 31 2025, 2:23 AM
F18822697: D16211.id.diff
Oct 23 2025, 8:01 AM
F18821595: D16211.diff
Oct 23 2025, 12:45 AM
F18766074: D16211.diff
Oct 7 2025, 2:49 PM
Subscribers

Details

Summary

New tokens, slightly larger (18x18 vs 16x16). I think these all feel decent, I might tweak the thumbs icons a little more color-wise.

Test Plan

Use Tokens.

pasted_file (366×599 px, 25 KB)

Diff Detail

Repository
rP Phabricator
Branch
tokens (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12873
Build 16413: Run Core Tests
Build 16412: arc lint + arc unit

Event Timeline

chad retitled this revision from to New Tokens.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad edited the test plan for this revision. (Show Details)
chad added inline comments.
webroot/rsrc/image/sprite-tokens.png
1

haha. oh. I don't want to make two sizes anymore and only build retina images. hrm.

  • Add 1x icons proper like
epriestley edited edge metadata.

The thumbs up / thumbs down are a bit hard for me to see against a light background.

This revision is now accepted and ready to land.Jul 1 2016, 11:34 AM

I took one pass at re-coloring it and it was much better, but may need a second pass. Wanted to see it live for a bit.

This revision was automatically updated to reflect the committed changes.