-
Notifications
You must be signed in to change notification settings - Fork 26.1k
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
Improve next/script
documentation
#31062
Comments
This PR does the following to partially address #31062: - Expands the [Script Component](https://nextjs.org/docs/basic-features/script) page in the core documentation - Adds a `next/script` API reference page
Along these lines I think it would be nice to have clarity on certain behaviors. For example: what happens if you are using props as values in the dangerous innerHTML for a Script component? Will it re-render and run the code again? What happens if you need multiple scripts with the same strategy to load in a specific order? Etc. |
This PR does the following to partially address vercel#31062: - Expands the [Script Component](https://nextjs.org/docs/basic-features/script) page in the core documentation - Adds a `next/script` API reference page
The Next.js Learn course now has a Third-Party JavaScript section that educates on Calling this ok for now. |
This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you. |
Describe the feature you'd like to request
There has been feedback that the usage of
next/script
can be difficult to understand for Next.js developers looking to optimize loading of third-party scripts. This is a multi-step proposal to improve how the Script component is explained in the documentation.Describe the solution you'd like
next/script
documentation #31063):next/script
in more detail (onLoad
, inline scripts, etc...)next/script
page to the API reference documentation (Expandsnext/script
documentation #31063)Describe alternatives you've considered
N/A
The text was updated successfully, but these errors were encountered: