-
Notifications
You must be signed in to change notification settings - Fork 6
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
fix(list): performance issue in list items are now resolved #417
Conversation
PR showcase deployed here. |
960e6d0
to
afe1d69
Compare
Avoids overhead in Safari + copes with the CRP 50ms task time
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand that you're currently trying to work this out directly with Material. Do you think it's worth waiting for their response?
I'm concerned that applying Material bug patches to our code will lead to confusion, and might itself cause issues now or down the road when Material closes this.
Having said that: If the performance problem handled here is extremely grave and common, we might wish to apply it. What are you saying? How would you classify the severity here?
It is currently a blocker with Vonage.ai integartion, so yes. We should go with this one. |
Related issue: material-components/material-web#1927