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
F22542263: D18048.diff
Tue, Sep 22, 4:43 PM
Unknown Object (File)
Apr 11 2026, 11:59 PM
Unknown Object (File)
Apr 10 2026, 10:43 PM
Unknown Object (File)
Apr 10 2026, 8:12 AM
Unknown Object (File)
Mar 30 2026, 8:31 PM
Unknown Object (File)
Mar 29 2026, 1:40 PM
Unknown Object (File)
Mar 4 2026, 1:02 PM
Unknown Object (File)
Feb 11 2026, 10:46 AM
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
Branch
head4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17320
Build 23210: Run Core Tests
Build 23209: arc lint + arc unit

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 43407

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