Page MenuHomePhabricator

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

Authored by epriestley on May 30 2017, 9:41 PM.
Tags
None
Referenced Files
F13979105: D18048.id43406.diff
Sat, Oct 19, 2:42 AM
F13956717: D18048.id43407.diff
Mon, Oct 14, 6:51 AM
Unknown Object (File)
Sep 30 2024, 5:48 PM
Unknown Object (File)
Sep 25 2024, 7:01 AM
Unknown Object (File)
Sep 5 2024, 1:28 AM
Unknown Object (File)
Sep 4 2024, 4:17 AM
Unknown Object (File)
Sep 2 2024, 10:38 AM
Unknown Object (File)
Sep 2 2024, 10:38 AM
Subscribers
None

Details

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

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

This revision is now accepted and ready to land.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 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"
This revision was automatically updated to reflect the committed changes.