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] Select API default value for input prop #8385

Merged
merged 2 commits into from Sep 25, 2017

Conversation

kgregory
Copy link
Member

The markdown was being rendered as an actual input field on the doc site and not at all when viewed on github. Left angle bracket needs to be < (as mentioned on gruber's syntax docs)

The markdown was being rendered as an actual input field - left angle bracket needed to be `<`
@oliviertassinari
Copy link
Member

🤣

capture d ecran 2017-09-25 a 17 17 30

@@ -10,7 +10,7 @@
| MenuProps | Object | | Properties applied to the `Menu` element. |
| <span style="color: #31a148">children *</span> | ChildrenArray | | The option elements to populate the select with. Can be some `MenuItem` when `native` is false and `option` when `native` is true. |
| classes | Object | | Useful to extend the style applied to components. |
| input | Element | <Input /> | An `Input` element. |
| input | Element | &lt;Input /> | An `Input` element. |
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll take care of it when parsing the default value

@oliviertassinari oliviertassinari added PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI docs Improvements or additions to the documentation v1 labels Sep 25, 2017
…eplace left angle bracket with &lt;

- Select input prop
- Checkbox indeterminateIcon
@oliviertassinari oliviertassinari removed the PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI label Sep 25, 2017
@oliviertassinari oliviertassinari merged commit a8d5fca into v1-beta Sep 25, 2017
@oliviertassinari oliviertassinari deleted the kgregory-patch-2 branch September 25, 2017 16:39
@oliviertassinari
Copy link
Member

👌✨

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants