Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[material-ui][Autocomplete] Improve design when there's a start adornment for small autocomplete (@TahaRhidouani) #42176

Merged
merged 1 commit into from
May 9, 2024
Merged

[material-ui][Autocomplete] Improve design when there's a start adornment for small autocomplete (@TahaRhidouani) #42176

merged 1 commit into from
May 9, 2024

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented May 9, 2024

Cherry-pick of #41781

@TahaRhidouani @ZeeshanTamboli @web-flow
[material-ui][Autocomplete] Improve design when there's a start adorn…
0928812 
…ment for small autocomplete (#41781)

Co-authored-by: ZeeshanTamboli <zeeshan.tamboli@gmail.com>
@github-actions github-actions bot added cherry-pick The PR was cherry-picked from the next branch component: autocomplete This is the name of the generic UI component, not the React module! design This is about UI or UX design, please involve a designer needs cherry-pick The PR should be cherry-picked to master after merge package: material-ui Specific to @mui/material labels May 9, 2024
@ZeeshanTamboli ZeeshanTamboli removed the needs cherry-pick The PR should be cherry-picked to master after merge label May 9, 2024
@mui-bot
Copy link

mui-bot commented May 9, 2024 •
edited
Loading

Netlify deploy preview

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

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against 0928812

@ZeeshanTamboli ZeeshanTamboli reopened this May 9, 2024
@ZeeshanTamboli ZeeshanTamboli merged commit c44cb3f into master May 9, 2024
20 checks passed
@ZeeshanTamboli ZeeshanTamboli deleted the cherry-pick-41781 branch May 9, 2024 10:11
@DiegoAndai
Copy link
Member

I'm not sure it was necessary to cherry-pick this 🤔 We should only cherry-pick urgent fixes.

Was there a particular reason to do it?

@ZeeshanTamboli
Copy link
Member

I'm not sure it was necessary to cherry-pick this 🤔 We should only cherry-pick urgent fixes.

Was there a particular reason to do it?

The user is using Material UI version 5. I wasn't aware that only urgent fixes, like regressions, are allowed to be cherry-picked. I think we should continue to support fixes on v5 at least until v6 is close to a stable release (maybe by the end of the beta phase?). Developers won't be migrating so early in production applications. From my experience working on GitHub issues, I've seen developers still migrating from v4 to v5. What are your thoughts?

@DiegoAndai
Copy link
Member

I think we should continue to support fixes on v5 at least until v6 is close to a stable release

Yeah, you're right. Let's go with that.

@nico-martinucci nico-martinucci mentioned this pull request Jun 3, 2024
Closed
This was referenced Jun 6, 2024
Open
Open
@mtrq-capco mtrq-capco mentioned this pull request Jun 6, 2024
Open
This was referenced Jun 6, 2024
Open
Open
This was referenced Jun 6, 2024
Open
Open
This was referenced Jun 6, 2024
Merged
Merged
This was referenced Jun 6, 2024
Merged
Merged
@mattboddy47 mattboddy47 mentioned this pull request Jun 6, 2024
Open
@aqibawan12 aqibawan12 mentioned this pull request Jun 6, 2024
Open
@venkatoriga venkatoriga mentioned this pull request Jul 17, 2024
Closed
This was referenced Jul 17, 2024
Merged
Merged
This was referenced Jul 17, 2024
Open
Open
@touchvignesh touchvignesh mentioned this pull request Jul 17, 2024
Open
This was referenced Jul 17, 2024
Open
Open
Open
Open
Open
@Crushoverride007 Crushoverride007 mentioned this pull request Jul 18, 2024
Open
@DanneelsSophie DanneelsSophie mentioned this pull request Jul 18, 2024
Open
This was referenced Jul 18, 2024
Open
Open
This was referenced Jul 18, 2024
Open
Open
This was referenced Jul 18, 2024
Closed
Closed
@snyk-io snyk-io bot mentioned this pull request Jul 18, 2024
Open
This was referenced Jul 19, 2024
Open
Open
Open
@Chanuth-silva10 Chanuth-silva10 mentioned this pull request Jul 19, 2024
Open
@FaroukAmr FaroukAmr mentioned this pull request Jul 19, 2024
Open
@DanneelsSophie DanneelsSophie mentioned this pull request Jul 19, 2024
Open
@aqibawan12 aqibawan12 mentioned this pull request Jul 19, 2024
Open
@FrancescoCoding FrancescoCoding mentioned this pull request Jul 19, 2024
Closed
@Poseidon-G Poseidon-G mentioned this pull request Jul 19, 2024
Open
@patrickrizzardi patrickrizzardi mentioned this pull request Jul 19, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ZeeshanTamboli ZeeshanTamboli ZeeshanTamboli approved these changes

Assignees
No one assigned
Labels
cherry-pick The PR was cherry-picked from the next branch component: autocomplete This is the name of the generic UI component, not the React module! design This is about UI or UX design, please involve a designer package: material-ui Specific to @mui/material
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mui-bot @DiegoAndai @ZeeshanTamboli @TahaRhidouani