-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
Add reference for named slot functions #8946
Conversation
✅ Deploy Preview for astro-docs-2 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
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 looks great, @BryceRussell ! Thank you so much for tackling this! See my comment below re: can we make advice easier if we suggest that the tag should be a standard HTML element.
I get that the naming of the tag is arbitrary, but I wonder whether presenting it as an element avoids some of the naming restrictions we don't want them to get tripped up by?
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
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.
Looks great, thanks for this @BryceRussell ! Merge whenever you're happy with it!
Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com>
…pi-reference.mdx` (#8944) * i18n(fr): Add `reference/api-refenrece.mdx` L0 to L501 now, miss L502 to L1026 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * i18n(fr): Add `reference/api-refenrece.mdx` L502 to L1026 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * i18n(fr): Fix links Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * i18n(fr): translate api-reference.mdx from l1026 to end * i18n(fr): Fix links, remove "NBSP" space and fix typo at "régionaux" L1570 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * i18n(fr): Update `reference/dev-toolbar-reference.mdx` from #8923 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * i18n(fr): Update `guides/routing.mdx` from #8941 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * Update src/content/docs/fr/reference/api-reference.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/guides/routing.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Update src/content/docs/fr/guides/routing.mdx Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * Updating file since #8946 Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> * i18n(fr): Fix typo in from #8947 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * i18n(fr): Update `guides/routing.mdx` from #8967 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * i18n(fr): Update `reference/api-reference.mdx` from #8967 Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> * chore: Mettre à jour la documentation de référence de l'API --------- Signed-off-by: Thomas Bonnet <thomasbnt@protonmail.com> Co-authored-by: Armand Philippot <git@armand.philippot.eu> Co-authored-by: Armand Philippot <59021693+ArmandPhilippot@users.noreply.github.com> Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
Description (required)
This PR partially addresses issue #8855 by adding an example for named slot functions to the reference page for
Astro.slots.render()
Related issues & labels (optional)