Skip to content

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

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

mnajdova merged 3 commits into
mui:masterfrom
bencevoros:autocomplete-existing-option-highlight-fix-2

Conversation

@bencevoros

@bencevoros bencevoros commented Apr 25, 2023 •

Copy link
Copy Markdown
Contributor

@mui-bot

mui-bot commented Apr 25, 2023 •

Copy link
Copy Markdown

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
Copy Markdown

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

@michaldudak michaldudak left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
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