Page MenuHomePhabricator

Make status icons centered in PHUI object list view
ClosedPublic

Authored by hach-que on Jul 8 2015, 2:01 AM.
Tags
None
Referenced Files
F13406752: D13592.id32845.diff
Fri, Jul 5, 10:10 PM
F13398924: D13592.diff
Thu, Jul 4, 8:41 AM
F13387830: D13592.id32847.diff
Mon, Jul 1, 9:07 AM
F13383975: D13592.diff
Sun, Jun 30, 1:38 PM
F13370401: D13592.diff
Thu, Jun 27, 5:34 PM
F13368476: D13592.id32849.diff
Thu, Jun 27, 6:38 AM
F13367890: D13592.id32848.diff
Thu, Jun 27, 3:56 AM
F13362055: D13592.id32849.diff
Tue, Jun 25, 5:48 PM
Subscribers

Details

Summary

Fixes T8789. This makes status icons centered in the PHUI object list view.

Test Plan

Looked at it.

Diff Detail

Repository
rP Phabricator
Branch
phui-fix
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7197
Build 7438: [Placeholder Plan] Wait for 30 Seconds
Build 7437: arc lint + arc unit

Event Timeline

hach-que retitled this revision from to Make status icons centered in PHUI object list view.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: chad.

I am a CSS noob and probably did this wrong

its not even the right file.

i swear i can hardly breathe right now.

I'm glad to have provided some entertainment then.

But in all seriousness, I tried search-results.css first, and that didn't seem to apply to pages that had search results on it... given that I was changing how phui-icon-view appears, phui-icon-view.css seemed to be the next most appropriate file to put it in.

Is there a better file that this should be placed in?

hach-que edited edge metadata.

Put it in the right file

webroot/rsrc/css/phui/phui-object-item-list-view.css
79

I'd move the block up under here since they go together. Other than that, works for me! Appreciate the help.

chad edited edge metadata.
This revision is now accepted and ready to land.Jul 8 2015, 2:30 AM
This revision was automatically updated to reflect the committed changes.