Skip to content

[material-ui] Add target property to buttons#11705

Merged
vvakame merged 1 commit into
DefinitelyTyped:masterfrom
ligaz:patch-8
Oct 16, 2016
Merged

[material-ui] Add target property to buttons#11705
vvakame merged 1 commit into
DefinitelyTyped:masterfrom
ligaz:patch-8

Conversation

@ligaz

@ligaz ligaz commented Oct 4, 2016

Copy link
Copy Markdown
Contributor

Buttons' target is passed to the underlying EnhancedButton properties: https://github.com/callemall/material-ui/blob/master/src/internal/EnhancedButton.js#L330

Bonus: Refactor the href property to be available in the base class.

Button's target is passed to the underlying EnhancedButton properties: https://github.com/callemall/material-ui/blob/master/src/internal/EnhancedButton.js#L330

Bonus: Refactor the `href` property to be available in the base class.
@dt-bot

dt-bot commented Oct 4, 2016

Copy link
Copy Markdown
Member

material-ui/material-ui.d.ts

to authors (@ngbrown @herrmanno). Could you review this PR?
👍 or 👎?

Checklist

  • pass the Travis CI test?

@ligaz

ligaz commented Oct 12, 2016

Copy link
Copy Markdown
Contributor Author

@vvakame What's the state of this PR?

@vvakame

vvakame commented Oct 12, 2016

Copy link
Copy Markdown
Member

oh, sorry!
We can't get response from author. please ping me after 3 days. I'll merge this PR.

@mxl

mxl commented Oct 15, 2016

Copy link
Copy Markdown
Contributor

@vvakame Please merge.

@vvakame
vvakame merged commit c180768 into DefinitelyTyped:master Oct 16, 2016
@mxl mxl mentioned this pull request Oct 16, 2016
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.

4 participants