Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[docs] Add Form Control Label Position TypeScript demo #15426

Closed

Conversation

donigianrp
Copy link
Contributor

@donigianrp donigianrp commented Apr 20, 2019

#14897

@mui-pr-bot
Copy link

mui-pr-bot commented Apr 20, 2019

No bundle size changes comparing bdaadb5...ab31a5b

Generated by 🚫 dangerJS against ab31a5b

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation typescript labels Apr 20, 2019
@@ -5,7 +5,7 @@ import { FormGroupProps, FormGroupClassKey } from '../FormGroup';
export interface RadioGroupProps
extends StandardProps<FormGroupProps, RadioGroupClassKey, 'onChange'> {
name?: string;
onChange?: (event: React.ChangeEvent<{}>, value: string) => void;
onChange?: (event: React.ChangeEvent<HTMLInputElement>, value: string) => void;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it look good to you @eps1lon? I don't understand why {} is used.

@eps1lon eps1lon self-requested a review April 22, 2019 05:50
@eps1lon
Copy link
Member

eps1lon commented Apr 22, 2019

Someone opened already a pull request for this one a day earlier #15408. Thank you for working on this still.

@eps1lon eps1lon closed this Apr 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants