[TextField] Migrate FilledInput to emotion#24617
[TextField] Migrate FilledInput to emotion#24617duganbrett wants to merge 5 commits intomui:nextfrom duganbrett:migrate-filled-input
Conversation
|
@material-ui/core: parsed: +0.14% , gzip: +0.17% |
|
Very open to guidance on this one. Some of the way classes need to be applied on the input variants is not that obvious to me. |
This was a bit trickier than what I anticipated :) I tried to fix all issues with a73218b (let's see if everything is resolved) We need to be careful when styling existing component regarding which things are coming from state (for example Regarding the overrides & classes, I belive we should add only the things which are new to this component, in this case the new names for the slots ( |
|
This one does look tricky. I'm not sure about the increase of specificity. Should we implement the |
That is the other option, yeah. Let me prepare a different PR for it and see which one will work better 👍 |
|
@oliviertassinari prepared #24634 I think it's better if we use |
|
@mnajdova @oliviertassinari Thank you, both! Learning a lot. I hope this helps others with the remaining TextField variants. Feel free to close. |
Thanks for exploring 🙏 , this was a tricky one. Hopefully now #24634 can serve as a template for the other input components. Closing this one. |
This PR migrates the FilledInput component to the new emotion format as a part of #24405.