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
[core] Batch small changes #21553
[core] Batch small changes #21553
Conversation
oliviertassinari
commented
Jun 23, 2020
- [docs] Explain why we make each BC a15528e
- [docs] Simpler redirection cleaning logic 9eec519: I don't think that we can know, ahead of time when the 301 are safe to be removed. It seems simpler to add the redirection in sequential order and use git blame to know when they were added.
- [docs] Fix typos be84d9f: Reported by https://www.triplechecker.com/site/material-ui-com
ffbc514
to
77e7926
Compare
Co-authored-by: Matt <github@nospam.33m.co>
Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
Co-authored-by: Matt <github@nospam.33m.co>
4320e8c
to
7882405
Compare
@@ -21,7 +21,7 @@ This component can be useful in a variety of situations: | |||
|
|||
## Frame deferring | |||
|
|||
In it's core, the NoSsr component purpose is to **defer rendering**. | |||
At its core, the NoSsr component's purpose is to **defer rendering**. |
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.
@mbrookes are you happy with the bolded text?
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.
The wording, or the fact that it's bold? Arguably it isn't "deferred" on the server, but "disabled". However I can find other examples of this usage, so it's probably okay.