Skip to content

fix(Button): only default to button when click is handled#383

Merged
eddywashere merged 1 commit intoreactstrap:masterfrom
TheSharpieOne:fix/default-button-type-onclick
Apr 3, 2017
Merged

fix(Button): only default to button when click is handled#383
eddywashere merged 1 commit intoreactstrap:masterfrom
TheSharpieOne:fix/default-button-type-onclick

Conversation

@TheSharpieOne
Copy link
Copy Markdown
Member

@TheSharpieOne TheSharpieOne commented Mar 31, 2017

fixes: #382

<Button>Go!</Button> -> <button class="btn btn-secondary">Go!</button>
<Button onClick={this.myHandler}>Go!</Button> -> <button type="button" class="btn btn-secondary">Go!</button>
<Button type="submit" onClick={this.myHandler}>Go!</Button> -> <button type="submit" class="btn btn-secondary">Go!</button>

@eddywashere
Copy link
Copy Markdown
Member

very reasonable fix, thanks!

@eddywashere eddywashere merged commit af3ccbe into reactstrap:master Apr 3, 2017
@TheSharpieOne TheSharpieOne deleted the fix/default-button-type-onclick branch October 3, 2017 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<Button> type attribute should default to 'submit', not 'button'

2 participants