Skip to content

[Autocomplete] Fixed autocomplete's existing option selection#37012

Merged
mnajdova merged 3 commits intomui:masterfrom
bencevoros:autocomplete-existing-option-highlight-fix-2
Jun 21, 2023
Merged

[Autocomplete] Fixed autocomplete's existing option selection#37012
mnajdova merged 3 commits intomui:masterfrom
bencevoros:autocomplete-existing-option-highlight-fix-2

Conversation

@bencevoros
Copy link
Contributor

@bencevoros bencevoros commented Apr 25, 2023

@mui-bot
Copy link

mui-bot commented Apr 25, 2023

Netlify deploy preview

https://deploy-preview-37012--material-ui.netlify.app/

Bundle size report

Details of bundle changes

Generated by 🚫 dangerJS against 13e6d20

@zannager zannager added the scope: autocomplete Changes related to the autocomplete. This includes ComboBox. label Apr 25, 2023
@zannager zannager requested a review from michaldudak April 25, 2023 14:29
@harrisonlo
Copy link

This also solves another bug I was going to submit where auto highlight would disappear if items length changes. Can we have this merged please?

@ZeeshanTamboli ZeeshanTamboli requested a review from sai6855 June 5, 2023 13:33
Copy link
Member

@michaldudak michaldudak left a comment

Choose a reason for hiding this comment

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

This looks good to me! Good job, @bencevoros!
Could you merge in the latest master before we close this to ensure all the tests are run on the current codebase?

@mnajdova mnajdova merged commit fe69429 into mui:master Jun 21, 2023
@mnajdova
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: autocomplete Changes related to the autocomplete. This includes ComboBox.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Autocomplete] the first option is not highlighted for autoHighlight using filterOptions

6 participants