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
|
ArmandPhilippot
left a comment
There was a problem hiding this comment.
I fixed a few links, feel free to double check that this was the correct resource. I think there are still some broken links.
Other than that (and a typo), my comments are mostly related to the formatting of Type: and <Since />. We have updated the api-reference.mdx file to make the formatting of this information more consistent (see #8943, the format reference is the one used in configuration-reference.mdx):
<Since />andType:should be inside the same paragraph- If a default value exists, a
Default:line should be added.
So I added some suggestions on this. I didn't double check each function. If you think of a missing Default: don't hesitate to add it in the same block! 😄
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
|
Thank you @ArmandPhilippot for that great consistency work! So appreciated! 💪 I think it's time for @martrapp to have a look at what we've all been up to.... 👀 |
ArmandPhilippot
left a comment
There was a problem hiding this comment.
It seems I missed two incorrectly formatted Type: in the previous review. 👀
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
|
Noting that the link should work after the new View Transitions docs have merged tomorrow! |
martrapp
left a comment
There was a problem hiding this comment.
Wow! My wholehearted thanks to the Arcane Tower 🧙🏼♂️ and everyone who helped. I've just added a few small suggestions that you may or may not want to consider.
| **Type:** `() => Promise<void>` | ||
| </p> | ||
|
|
||
| Implementation of the following phase in the navigation (loading the next page). This implementation can be overriden to add extra behavior or to prevent the execution of the next phase. |
There was a problem hiding this comment.
... or to prevent the execution of the next phase
That is a mystery to me. Do we have any examples of this anywhere? Why would you do that? And why does it cancel the next phase? Calling preventDefault() for the astro:before-preparation event, on the other hand, is fine.
If we do not persuade users of this, we do not need to warn them in the following
There was a problem hiding this comment.
I haven't seen an example, nor can I think of any particular use case where you'd want to override this and not call the original at some point in the new function, but technically, you can.
I'm fine with not mentioning it on the docs tho.
|
Just resolved conflicts/updated branch! With any luck, that means the link check now passes! |
Co-authored-by: Martin Trapp <94928215+martrapp@users.noreply.github.com>
|
Needed to resolve conflicts again because this page just updated! 😂 |
sarah11918
left a comment
There was a problem hiding this comment.
Thank you so much @Fryuni for taking the initiative here! Would not be possible without the outstanding contributions of @ArmandPhilippot and the wise oversight of @martrapp .
Best Docs Team Ever! 🥳
yanthomasdev
left a comment
There was a problem hiding this comment.
Good work! I have mostly nit and typo suggestions here, after applying them I believe it's good to go!
Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
#9260) * i18n(fr): add Actions in `api-reference.mdx` See #9224 and #9254 * i18n(fr): add View Transitions section in api-reference.mdx See #9174 * i18n(fr): improve Actions handler documentation See #9261 * i18n(fr): update `view-transitions.mdx` See #9224 * i18n(fr): reword sentences around handler to make reading easier * i18n(fr): fix View Transitions capitalization where legit Co-authored-by: Thomas Bonnet <thomasbnt@protonmail.com> --------- Co-authored-by: Thomas Bonnet <thomasbnt@protonmail.com> Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
Description
Add a reference to the APIs and events available when using View Transitions.
Related issues & labels (optional)