-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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 limitTags prop #20209
[Autocomplete] Add limitTags prop #20209
Conversation
@material-ui/lab: parsed: +0.14% , gzip: +0.19% Details of bundle changes.Comparing: 7e1da61...fee1583 Details of page changes
|
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.
Great start
ebdf4b4
to
a618f46
Compare
f29cd03
to
a0aa08d
Compare
4c6e446
to
5d16990
Compare
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.
@netochaves I have applied the following
- I have rebased on master to solve the conflict
- I have removed "more" from the built-in message to avoid any translation concerns
- I have applied the tag class name to the limit label to ensure proper spacing
- I have added a function to customize the limit label
- I have renamed the props to be shorter and more descriptive
- I have simplified the test case
If you could double check the changes, it would be awesome. feedback welcome
5d16990
to
fb87c9b
Compare
fb87c9b
to
fee1583
Compare
Thanks for these improvements @oliviertassinari. I had some troubles with prettier because of this commit #20211, sorry about that. I really liked the |
Great, let's move forward :) |
Thanks for the improvement! I would like to use this but when i install the npm package it seems to be missing.. Am i doing something wrong? Do i need to wait for some time? |
Yes, I also don't see it in the latest package available on NPM. |
Closes #19137
This PR adds a new prop to handle the amount of items visible without focus.