Page MenuHomePhabricator
Differential D18032

Try layering state icons on PHUICircleIconView
ClosedPublic
Actions

Authored by chad on May 26 2017, 6:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:19 AM
Unknown Object (File)
Sat, Apr 13, 9:35 PM
Unknown Object (File)
Sat, Apr 13, 6:30 PM
Unknown Object (File)
Sat, Apr 13, 5:18 PM
Unknown Object (File)
Sat, Apr 13, 9:35 AM
Unknown Object (File)
Sat, Apr 13, 8:02 AM
Unknown Object (File)
Sat, Apr 13, 1:25 AM
Unknown Object (File)
Thu, Apr 11, 8:51 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP81809713e078: Try layering state icons on PHUICircleIconView
Summary

I think this is reasonable for my current use case, but stacking icons overally is pretty clunky.

Test Plan

UIExamples

image.png (96×1 px, 16 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.May 26 2017, 6:43 PM
Herald added a subscriber: Korvin. · View Herald TranscriptMay 26 2017, 6:43 PM
chad edited the test plan for this revision. (Show Details)May 26 2017, 6:44 PM
Harbormaster completed remote builds in B17277: Diff 43367.May 26 2017, 6:44 PM
epriestley accepted this revision.May 26 2017, 8:11 PM

Looks pretty good to me.

This revision is now accepted and ready to land.May 26 2017, 8:11 PM
Closed by commit rP81809713e078: Try layering state icons on PHUICircleIconView (authored by chad, committed by chad). · Explain WhyMay 26 2017, 8:55 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
applications/
uiexample/
examples/
17 lines
view/
phui/
47 lines
webroot/
rsrc/
css/
phui/
34 lines

Diff 43378

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/uiexample/examples/PHUIIconExample.php

Loading...
View Options

src/view/phui/PHUIIconCircleView.php

Loading...
View Options

webroot/rsrc/css/phui/phui-icon.css

Loading...
Phabricator · Built by Phacility