Page MenuHomePhabricator

New Tokens
ClosedPublic

Authored by chad on Jul 1 2016, 4:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 4, 6:20 PM
Unknown Object (File)
Wed, Apr 24, 10:27 PM
Unknown Object (File)
Sat, Apr 20, 3:23 PM
Unknown Object (File)
Thu, Apr 11, 7:34 AM
Unknown Object (File)
Wed, Apr 10, 5:36 AM
Unknown Object (File)
Apr 4 2024, 3:23 PM
Unknown Object (File)
Apr 3 2024, 12:57 PM
Unknown Object (File)
Apr 3 2024, 12:57 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 12874
Build 16415: Run Core Tests
Build 16414: 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.