Page MenuHomePhabricator
Differential D18048

Make "simple" a "button type", not a "color"
ClosedPublic
Actions

Authored by epriestley on May 30 2017, 9:41 PM.
Tags
None
Referenced Files
F15431461: D18048.id43419.diff
Mon, Mar 24, 12:28 PM
F15428820: D18048.id.diff
Sun, Mar 23, 10:35 PM
F15428281: D18048.id43407.diff
Sun, Mar 23, 6:52 PM
F15423968: D18048.id43406.diff
Sat, Mar 22, 7:42 PM
F15423312: D18048.diff
Sat, Mar 22, 2:41 PM
F15385683: D18048.id43406.diff
Fri, Mar 14, 11:10 PM
F15384788: D18048.id43419.diff
Fri, Mar 14, 8:50 PM
F15378569: D18048.id43407.diff
Thu, Mar 13, 3:21 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
rP2c0dab055f73: Make "simple" a "button type", not a "color"
Summary

Ref M1476. Currently, setColor('simple') is meaningful. Instead, setButtonType('simple').

Depends on D18047.

Test Plan

Looked at UI examples, Phame, Auth. Notifications mooted by D18047.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 30 2017, 9:41 PM
epriestley added a comment.May 30 2017, 9:42 PM

Open to other word choice here, but setStyle() sounds like HTML style="..." attributes.

Harbormaster completed remote builds in B17319: Diff 43406.May 30 2017, 9:43 PM
chad accepted this revision.May 30 2017, 9:45 PM
This revision is now accepted and ready to land.May 30 2017, 9:45 PM
chad added a comment.May 30 2017, 9:45 PM

I think we usually do "setType" but I hate not being able to easily grep that.

Alternative: setButtonType

epriestley updated this revision to Diff 43407.May 30 2017, 9:50 PM
epriestley retitled this revision from Make "simple" a "design", not a "color" to Make "simple" a "button type", not a "color".
epriestley edited the summary of this revision. (Show Details)
  • "Design" -> "Button Type"
Harbormaster completed remote builds in B17320: Diff 43407.May 30 2017, 9:51 PM
Closed by commit rP2c0dab055f73: Make "simple" a "button type", not a "color" (authored by epriestley, committed by epriestley). · Explain WhyMay 31 2017, 12:59 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
auth/
controller/
config/
2 lines
guides/
view/
2 lines
phame/
query/
2 lines
uiexample/
examples/
10 lines
4 lines
8 lines
view/
phui/
22 lines
2 lines

Diff 43419

View Options

src/applications/auth/controller/config/PhabricatorAuthListController.php

Loading...
View Options

src/applications/guides/view/PhabricatorGuideListView.php

Loading...
View Options

src/applications/phame/query/PhameBlogSearchEngine.php

Loading...
View Options

src/applications/uiexample/examples/PHUIBoxExample.php

Loading...
View Options

src/applications/uiexample/examples/PHUIButtonBarExample.php

Loading...
View Options

src/applications/uiexample/examples/PHUIButtonExample.php

Loading...
View Options

src/view/phui/PHUIButtonView.php

Loading...
View Options

src/view/phui/PHUIDocumentViewPro.php

Loading...
Phabricator · Built by Phacility