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

[styled] Fix override logic to support component without root #25143

Merged
merged 4 commits into from
Mar 1, 2021

Conversation

niting143
Copy link
Contributor

@niting143 niting143 commented Feb 28, 2021

Closes #25141

@mui-pr-bot
Copy link

mui-pr-bot commented Feb 28, 2021

Details of bundle changes

Generated by 🚫 dangerJS against 750b774

@oliviertassinari oliviertassinari marked this pull request as draft February 28, 2021 20:14
@niting143 niting143 marked this pull request as ready for review February 28, 2021 20:24
@oliviertassinari oliviertassinari changed the title Style overrides [styled] Fix override logic to support component without root Feb 28, 2021
@oliviertassinari oliviertassinari added the bug 🐛 Something doesn't work label Mar 1, 2021
@oliviertassinari oliviertassinari merged commit 23bfe37 into mui:next Mar 1, 2021
@oliviertassinari
Copy link
Member

@niting143 I went ahead and added two test cases once for the tooltip, which is really hard to debug when it fails, and another one with a shorter feedback loop. This change will allow fixing #25147 as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Tooltip] styleOverrides not working
3 participants