-
-
Notifications
You must be signed in to change notification settings - Fork 31.6k
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
[docs] Use demo name as codesandbox name #22926
Conversation
<link rel="stylesheet" href="https://fonts.googleapis.com/icon?family=Material+Icons" /> | ||
<div id="root"></div> | ||
</body> | ||
'public/index.html': ` |
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.
See codesandbox/codesandbox-client#4737. Old sandboxes are fine since they only use body
. Once we want to use head
we need to have it in public/index.html
.
Benefit: Now also works when downloaded though we've never heard anyone complain.
No bundle size changes comparing d08a755...f991de4 |
I'll leave this open till the end of the week in case we get a response to codesandbox/codesandbox-client#4983. |
Includes the demo name in the codesandbox and page title. Also includes the currently used docs language for more accurate repros.
Note that this only works when the sandbox is opened in a new window. The integrated view has a non-descriptive UUID-like title. Possibly due to codesandbox/codesandbox-client#4983.