[material-ui][Button] Revert Button loading

[siriwatknp]

To add loading to the Button, it's required to wrap the text with another span element to prevent Google translation crash so it's a breaking change.

The work can be revisited in v7.

Revert #42987

[mui-bot]

Netlify deploy preview

• docs/data/material/components/button-group/button-group.md
• docs/data/material/components/buttons/buttons.md
• docs/data/material/migration/upgrade-to-v6/upgrade-to-v6.md

IconButton: parsed: -4.33% 😍, gzip: -3.45% 😍
Alert: parsed: -3.84% 😍, gzip: -2.94% 😍
Autocomplete: parsed: -2.38% 😍, gzip: -1.78% 😍
@material-ui/core: parsed: -0.58% 😍, gzip: -0.41% 😍
LoadingButton: parsed: +1.16% , gzip: +0.38%
@material-ui/lab: parsed: +0.14% , gzip: -0.02% 😍

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against 767a9bc

[michaldudak]

@zannager, since I don't work on Material UI, someone from the Core team might have better insights on the topic.

[aarongarciah]

😢

[oliviertassinari]

To add loading to the Button, it's required to wrap the text with another span element to prevent Google translation crash so it's a breaking change.

Is this about reverting #26666? It feels a bit like adding bloat, is Google translation + loading important enough?

What if we were to give up on this? #27853 suggests that it's important to fix. But maybe we could leave this as a documentation note? We could ship this now, and wait to see it works. In the docs, we could suggest:

1. to patch DOM operations Make React resilient to DOM mutations from Google Translate facebook/react#11538 (comment)
2. or add wrapping span Make React resilient to DOM mutations from Google Translate facebook/react#11538 (comment)
3. to disable translations: Make React resilient to DOM mutations from Google Translate facebook/react#11538 (comment)

https://martijnhols.nl/gists/everything-about-google-translate-crashing-react is an interesting dive into this problem.