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.

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

[TextField] Fix conflict with Bootstrap even when label is not defined #34343

Conversation

ZeeshanTamboli
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli commented Sep 17, 2022

Fixes #34298.

Even when label is not set in TextField, it should work correctly with bootstrap.

Similar issue #19463. It's PR - #19495.

I am not sure how width: auto works here. I just referred #19495.


Before: https://codesandbox.io/s/charming-maria-vk4q3l?file=/src/App.tsx

After: https://codesandbox.io/s/young-water-09u5r0?file=/src/App.tsx

@ZeeshanTamboli ZeeshanTamboli added the component: text field This is the name of the generic UI component, not the React module! label Sep 17, 2022
@mui-bot
Copy link

mui-bot commented Sep 17, 2022

Details of bundle changes

Generated by 🚫 dangerJS against 666d5db

@ZeeshanTamboli ZeeshanTamboli changed the title [TextField] Fix conflict with bootstrap even when label is not defined [TextField] Fix conflict with Bootstrap even when label is not defined Sep 17, 2022
@ZeeshanTamboli ZeeshanTamboli marked this pull request as ready for review September 17, 2022 10:23
@ZeeshanTamboli ZeeshanTamboli added the bug 🐛 Something doesn't work label Sep 17, 2022
@oliviertassinari oliviertassinari added the package: material-ui Specific to @mui/material label Sep 17, 2022
@ZeeshanTamboli ZeeshanTamboli merged commit f5efa42 into mui:master Sep 19, 2022
@ZeeshanTamboli ZeeshanTamboli deleted the issue/34298-textfield-legend-style-bootstrap branch September 19, 2022 08:37
@oliviertassinari
Copy link
Member

@ZeeshanTamboli This is a clean fix, thanks for the care.

@ZeeshanTamboli
Copy link
Member Author

@ZeeshanTamboli This is a clean fix, thanks for the care.

@oliviertassinari As usual, glad to be of help. :)

daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
…ned (mui#34343)

Co-authored-by: Michał Dudak <michal@dudak.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: text field This is the name of the generic UI component, not the React module! package: material-ui Specific to @mui/material
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TextField] legend style conflicts with Bootstrap when no label is set
4 participants