Page MenuHomePhabricator

New Tokens
ClosedPublic

Authored by chad on Jul 1 2016, 4:10 AM.
Tags
None
Referenced Files
F15433094: D16211.diff
Mon, Mar 24, 9:21 PM
F15387883: D16211.id39000.diff
Sat, Mar 15, 3:03 AM
F15385691: D16211.id38996.diff
Fri, Mar 14, 11:11 PM
F15354488: D16211.id.diff
Tue, Mar 11, 4:15 AM
Unknown Object (File)
Feb 27 2025, 12:50 PM
Unknown Object (File)
Feb 22 2025, 9:52 AM
Unknown Object (File)
Feb 16 2025, 7:03 AM
Unknown Object (File)
Feb 9 2025, 1:55 AM
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.