Skip to content

[Icon] Disable translation, e.g Google Translate#23237

Merged
oliviertassinari merged 4 commits into
mui:nextfrom
cbeltrangomez84:next
Oct 27, 2020
Merged

[Icon] Disable translation, e.g Google Translate#23237
oliviertassinari merged 4 commits into
mui:nextfrom
cbeltrangomez84:next

Conversation

@cbeltrangomez84

@cbeltrangomez84 cbeltrangomez84 commented Oct 25, 2020

Copy link
Copy Markdown
Contributor

@mui-pr-bot

mui-pr-bot commented Oct 25, 2020

Copy link
Copy Markdown

Details of bundle changes

Generated by 🚫 dangerJS against 6d44663

@cbeltrangomez84

cbeltrangomez84 commented Oct 25, 2020

Copy link
Copy Markdown
Contributor Author

I'm not sure why the test_browser-1 failed, help needed since I made a minor change only.

@oliviertassinari oliviertassinari changed the title Make the Icons avoid Google Chrome translation since it make it stop … [Icon] Disable translation, e.g Google Translate Oct 25, 2020
@oliviertassinari oliviertassinari added type: bug It doesn't behave as expected. component: Icon The React component. labels Oct 25, 2020
@mbrookes

Copy link
Copy Markdown
Member

@oliviertassinari

Copy link
Copy Markdown
Member

@mbrookes This is a different issue, the JavaScript doesn't load in your test case.

Comment thread packages/material-ui/src/Icon/Icon.js Outdated
Comment thread packages/material-ui/src/Icon/Icon.js Outdated
@cbeltrangomez84

Copy link
Copy Markdown
Contributor Author

Thanks @oliviertassinari and @eps1lon

Do I need to do something else? This is my first pull request here.

Comment thread packages/material-ui/src/Icon/Icon.js Outdated
@oliviertassinari
oliviertassinari merged commit d84ca55 into mui:next Oct 27, 2020
@oliviertassinari

Copy link
Copy Markdown
Member

@cbeltrangomez84 It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!

sujinleeme pushed a commit to sujinleeme/material-ui that referenced this pull request Oct 28, 2020
sujinleeme pushed a commit to sujinleeme/material-ui that referenced this pull request Oct 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: Icon The React component. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Google Translate on Chrome make the Icons display not correctly

5 participants