Page MenuHomePhabricator
Differential D9746

Slimmer Shade Tags
ClosedPublic
Actions

Authored by chad on Jun 26 2014, 9:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 15, 5:19 PM
Unknown Object (File)
Sun, Sep 15, 4:56 PM
Unknown Object (File)
Sun, Sep 15, 4:21 PM
Unknown Object (File)
Tue, Sep 3, 2:23 AM
Unknown Object (File)
Mon, Sep 2, 2:01 AM
Unknown Object (File)
Sun, Sep 1, 12:12 AM
Unknown Object (File)
Thu, Aug 29, 4:14 PM
Unknown Object (File)
Thu, Aug 22, 2:16 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPe096a227db1d: Slimmer Shade Tags
Summary

Shaves a pixel for use in ObjectLists.

Test Plan

UIExamples.

undefined (432×380 px, 29 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

chad updated this revision to Diff 23405.Jun 26 2014, 9:13 PM
chad retitled this revision from to Slimmer Shade Tags.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJun 26 2014, 9:13 PM
chad edited the test plan for this revision. (Show Details)Jun 26 2014, 9:14 PM
Harbormaster completed remote builds in B1417: Diff 23405.Jun 26 2014, 9:14 PM
epriestley edited edge metadata.Jun 26 2014, 9:17 PM

wow such savings!

epriestley accepted this revision.Jun 26 2014, 9:17 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 26 2014, 9:17 PM
chad closed this revision.Jun 26 2014, 9:19 PM
chad updated this revision to Diff 23406.

Closed by commit rPe096a227db1d (authored by @chad).

chad added a comment.Jun 26 2014, 9:22 PM

erm, wait did I just make this for nothing, looks like the obj list shades are already smaller?

epriestley added inline comments.Jun 26 2014, 9:26 PM
webroot/rsrc/css/phui/phui-tag-view.css
319–346

I have hard-coded rules down here which were my coarse pass at this, but the explicit slimming is better and your numbers are a little different (and presumably superior).

epriestley added a comment.Jun 26 2014, 9:27 PM

I'll shoot you a diff to swap over to this and toss out my rules, but I'm going to try Shovel Knight first.

shovel shovel shovel

chad added a comment.Jun 26 2014, 9:28 PM

Yes, but it's the same size as where I wanted to replace (ObjList) so it's not actually slimmer there.

Here is another pass (shaves another 2px:

undefined (228×826 px, 27 KB)

chad added a comment.Jun 26 2014, 9:28 PM

Oh I paid for that on Kickstarter, guess I should download.

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
applications/
uiexample/
examples/
8 lines
view/
phui/
10 lines
webroot/
rsrc/
css/
phui/
11 lines

Diff 23406

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/uiexample/examples/PHUITagExample.php

Loading...
View Options

src/view/phui/PHUITagView.php

Loading...
View Options

webroot/rsrc/css/phui/phui-tag-view.css

Loading...
Phabricator · Built by Phacility