docs icon indicating copy to clipboard operation
docs copied to clipboard

Revise hydration section

Open Jutanium opened this issue 2 years ago • 1 comments

What kind of changes does this PR include?

  • New or updated content

Description

  • Replace a Caution which made it seem like the component framework JavaScript was sent on page load
  • Add a client:only example to the code snippet, because we mention that "most directives" render the component on the server
  • Make the intro more concise

Jutanium avatar Oct 05 '22 20:10 Jutanium

Deploy Preview for astro-docs-2 ready!

Name Link
Latest commit d744f0737019c5d135d6bbc538c7914ff6b9d789
Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/634f19ce3c0f160007b7cd57
Deploy Preview https://deploy-preview-1728--astro-docs-2.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Oct 05 '22 20:10 netlify[bot]

Just replying to both comments, which are basically the same comment, that this PR as is makes things clearer, and I think it's fine to merge without figuring out how to deal with the pesky client:only. Remember, the heavy lifting of these descriptions is currently done on the Template Directives reference page. So the real mandate of this page is just to make people know that these exist, and to point them on to reference for more. (We may not always choose to keep this structure, but that's what we have right now, and it's OK to fall back on that.)

So, LGTM when you both are happy!

sarah11918 avatar Oct 18 '22 22:10 sarah11918

Lets Get This Merged! 🚀

kevinzunigacuellar avatar Oct 18 '22 22:10 kevinzunigacuellar