Page MenuHomePhabricator

Slimmer Shade Tags
ClosedPublic

Authored by chad on Jun 26 2014, 9:13 PM.
Tags
None
Referenced Files
F14035623: D9746.diff
Sun, Nov 10, 6:48 AM
F14017034: D9746.id23406.diff
Mon, Nov 4, 1:50 PM
F14008925: D9746.id23406.diff
Wed, Oct 30, 8:02 AM
F13987309: D9746.id23406.diff
Mon, Oct 21, 8:40 AM
F13967364: D9746.diff
Wed, Oct 16, 1:07 PM
F13966401: D9746.id23406.diff
Wed, Oct 16, 7:55 AM
Unknown Object (File)
Sep 25 2024, 4:06 AM
Unknown Object (File)
Sep 25 2024, 4:00 AM
Subscribers

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
Branch
slim-shady-css
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1417
Build 1417: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

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.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 26 2014, 9:17 PM
chad updated this revision to Diff 23406.

Closed by commit rPe096a227db1d (authored by @chad).

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

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).

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

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)

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