Skip to content

Conversation

@ManthanGajjar
Copy link
Contributor

@ManthanGajjar ManthanGajjar commented Jul 18, 2024

Fixes #42882

Preview: https://deploy-preview-42990--material-ui.netlify.app/material-ui/react-autocomplete/#hint

Issue description

the dropdown does not show the selected option after clicking on it

Before:

Screen.Recording.2024-07-18.at.3.38.22.PM.mov

After:

Screen.Recording.2024-07-18.at.3.38.41.PM.mov

@ManthanGajjar ManthanGajjar marked this pull request as draft July 18, 2024 10:12
@mui-bot
Copy link

mui-bot commented Jul 18, 2024

Netlify deploy preview

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 581074e

@ManthanGajjar ManthanGajjar reopened this Jul 18, 2024
@ManthanGajjar ManthanGajjar reopened this Jul 18, 2024
@ManthanGajjar ManthanGajjar marked this pull request as ready for review July 18, 2024 10:29
@ManthanGajjar
Copy link
Contributor Author

Not able to add / update labels in this PR :(

@zannager zannager added the scope: autocomplete Changes related to the autocomplete. This includes ComboBox. label Jul 18, 2024
@zannager zannager requested a review from sai6855 July 18, 2024 13:22
@ZeeshanTamboli ZeeshanTamboli added type: bug It doesn't behave as expected. docs Improvements or additions to the documentation. package: material-ui labels Jul 21, 2024
@ZeeshanTamboli ZeeshanTamboli changed the title [material] fix autocomplete hint issue [docs][material-ui][Autocomplete] Fix Hint demo Jul 21, 2024
@ZeeshanTamboli ZeeshanTamboli added the type: regression A bug, but worse, it used to behave as expected. label Jul 21, 2024
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli left a comment

Choose a reason for hiding this comment

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

This fix works well for selecting option from Autocomplete but I found other issues as well. See screen recording:

issue-42990.mp4

@ZeeshanTamboli ZeeshanTamboli added the package: joy-ui Specific to Joy UI. label Jul 23, 2024
@ZeeshanTamboli ZeeshanTamboli changed the title [docs][material-ui][Autocomplete] Fix Hint demo [docs][material-ui][joy-ui][Autocomplete] Fix Hint demo Jul 23, 2024
@ZeeshanTamboli ZeeshanTamboli added the needs cherry-pick The PR should be cherry-picked to master after merge. label Jul 23, 2024
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli left a comment

Choose a reason for hiding this comment

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

I fixed the overflowing hint text and also made sure that the hint text is cleared when a option as selected. Rest looks good. @ManthanGajjar Thanks for the pull request.

@ZeeshanTamboli ZeeshanTamboli merged commit 64a093a into mui:next Jul 23, 2024
github-actions bot pushed a commit that referenced this pull request Jul 23, 2024
joserodolfofreitas pushed a commit to joserodolfofreitas/material-ui that referenced this pull request Jul 29, 2024
@oliviertassinari oliviertassinari removed the type: bug It doesn't behave as expected. label Apr 30, 2025
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. needs cherry-pick The PR should be cherry-picked to master after merge. package: joy-ui Specific to Joy UI. scope: autocomplete Changes related to the autocomplete. This includes ComboBox. type: regression A bug, but worse, it used to behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs][material-ui][Autocomplete] Hint demo is broken

5 participants