Button: `type` is working incorrecly

[layershifter]

Bug Report

Hi guys, I'm building now some stuff based on Stardust specifications and I found the design problem.

However, the type prop is reserved in button: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/button

Steps

<Button content="Click here" type="reset" />

Expected Result

<button class="ui-button" aria-disabled="false" type="reset">
  <span class="ui-slot">Click here</span>
</button>

Actual Result

<button class="ui-button" aria-disabled="false">
  <span class="ui-slot">Click here</span>
</button>

Version

0.9.1

[hughreeling]

As far as I know there are several discussions indirectly about this. There need to be more clarify about if 'default' , 'primary' and 'secondary' really all need to exist... and if we shouldn't just use <Button content="click" primary />

…

On Fri, Oct 19, 2018 at 9:28 AM Alexander Fedyashov < ***@***.***> wrote: Bug Report Hi guys, I'm building now some stuff based on Stardust specifications and I found the design problem. [image: image] <https://user-images.githubusercontent.com/14183168/47203623-a199ed80-d389-11e8-9570-159189ccf561.png> However, the type prop is reserved in button: Steps <Button content="Click here" type="reset" /> Expected Result The result that you expected. Actual Result The actual result that happened 💣 Version x.y.z Testcase — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#374>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGjGS2skThE_AwR6wqDechxwtVKXDpk1ks5umX8ogaJpZM4Xv0IB> .

[levithomason]

+1 to boolean prop API for this