-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[Autocomplete] Add docs interface for the hook #36242
[Autocomplete] Add docs interface for the hook #36242
Conversation
…se-autocomplete-hook
…se-autocomplete-hook
The type was wrongly spelled, it fixes the generation. -AutoComplete
+Autocomplete |
Thanks for reviewing and fixing those up Olivier! Much appreciated |
…se-autocomplete-hook
value: AutocompleteValue<T, Multiple, DisableClearable, FreeSolo>; | ||
/** | ||
* If `true`, the component has valid values selected. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be:
* If `true`, the component has valid values selected. | |
* If `true`, the component input has some values. |
Because in freeSolo
mode, it can be any value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call-out, have amended my PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks.
Adds useAutocomplete return value interface
closes useAutocomplete in #35933
I was unable to generate any JS docs from runningyarn docs:api
so I probably missed exporting the new return value type somewhere, but I'm having some trouble figuring it out where that might be as the PR's for the other public hooks all had.ts
versions for the hook butuseAutocomplete
only exists in.js
so there must be a different mechanism. Pointers on this would be greatly appreciatedhttps://deploy-preview-36242--material-ui.netlify.app/base/api/use-autocomplete/#return-value