Skip to content

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Sep 18, 2024

See https://www.chromium.org/for-testers/faq/ for what happened.

I have noticed this while I was browsing the docs and curious about what a Chromium bug about text ellipse with flexbox was about. A quick win.

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation. label Sep 18, 2024
@mui-bot
Copy link

mui-bot commented Sep 18, 2024

@@ -4,8 +4,6 @@ const IconRoot = styled('span')(({
userSelect: 'none',
width: '1em',
height: '1em',
// Chrome fix for https://bugs.chromium.org/p/chromium/issues/detail?id=820541
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this not relevant anymore?

Copy link
Member Author

@oliviertassinari oliviertassinari Sep 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to keep track of the status of each of these bugs. Once they are fixed and propagated enough with end users, we should update our implementation to take advantage of those fixes. I think it's important to stay relevant.

Now, this is a codemod file, I think removing this comment will make it easier to maintain those.

@oliviertassinari oliviertassinari merged commit e333475 into mui:master Sep 19, 2024
22 of 23 checks passed
@oliviertassinari oliviertassinari deleted the docs-fix-301-chromium branch September 19, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants