-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add new “Scripts and Event Handling” guide #2102
Conversation
✅ Deploy Preview for astro-docs-2 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
This is an amazing page, @delucis! 🚀 I can't wait for it to go live!
I think there might only be 1 or 2 spots to really revisit and nail the explanations, but I've made some possible editing suggestions that are potentially better, if true! You'll see this before I will tomorrow, so some time for you to digest, correct me, and then we can tackle it tomorrow!
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
LGTM! And so begins the downfall of "Components"... (everything's a component) |
What kind of changes does this PR include?
Description
Simplifies the “Client-side scripts” section of the Components guide
Adds a new guide to writing
<scripts>
— https://deploy-preview-2102--astro-docs-2.netlify.app/en/guides/client-side-scripts/Includes guidance on:
<script>
define:vars
)Moves detailed content on script bundling and loading out of the Components guide to the new page
Closes No documentation about Web Components #2100
Useful things to answer
How does this feel? It’s a slightly 50-50 page with a reference-y first half and guide-y second half.
Are the examples clear and helpful? I tried to pick things I could distil down but also chose not to dive too deep into custom element syntax etc. as I think that probably is out of scope for the Astro docs.
Given that, are the links helpful? Can people find what they need to know by following them?
Nav bar position? I feel grouping this with both styling & UI frameworks makes sense. Does SSR need to move up/down?