-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
perf: load script only on pages where it is used #288
Comments
I think the plugin's functionality to load the script would need to be moved to the |
Yes, I think it would be nice to move it to the component, if we can implement it so it's deduplicated across components and also not removed from the head if the component is unmounted (as this could result in the script being loaded more than once). |
That's great to hear... Can you create a pull request for it? |
This comment was marked as spam.
This comment was marked as spam.
I think this is a good task for a community PR, help welcome! 🙌 |
@danielroe did #306 help to resolve this issue or is this unrelated? |
yes, this should be resolved! |
📚 Is your documentation request related to a problem?
By default the Script is being loaded on all pages. I want it to load on selected pages. How can I do that?,
🔍 Where should you find it?
No response
ℹ️ Additional context
No response
The text was updated successfully, but these errors were encountered: