Buttons will render as plain even if explicit told to not be plain

Expected Behavior

If I use the Button element like: <Button plain={ false } primary onClick={ func }>Text</Button> I expect that the button is told to be not plain.

Actual Behavior

Actually the button gets rendered with the grommetux-button–plain css class.

URL, screen shot, or Codepen exhibiting the issue

http://codepen.io/anon/pen/pNorYo?&editors=0010 The plain attribute seems to have no impact when using it with children (as it stands in the code).

Steps to Reproduce

1. Button with plain attribute
2. Button without plain attribute

Your Environment

• Grommet version: 1.0.1
• Browser Name and version: Chrome 54.0.2840.71 m
• Operating System and version (desktop or mobile): Windows 7

[ericsoderberghp]

My inclination is to deprecate the plain property all together. The Button controls styling if icon or label are set and doesn’t touch the styling if there are child components.

[ibiBgOR]

I think you guys are right. Deprecating the plain property sounds pretty logical. If the Button component takes all the style work from me I will not moan. 😄 Thank you for your help