update script behavior section

[OliverSpeir]

Description (required)

Reworks the Scripts section of view-transitions.mdx, relativey significantly. Adds documentation for new attribute of <script> tags data-astro-rerun

Related PR

withastro/astro#9977

For Astro version: 4.5

Would love your review on this @martrapp

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	Mar 9, 2024 7:02pm

[martrapp]

As for the broken link: It is mdx. You could replace the ## with an <h2> and the old id or insert an anchor with the old id right in front of the ##, see suggestion.

[OliverSpeir]

As for the broken link: It is mdx. You could replace the ## with an <h2> and the old id or insert an anchor with the old id right in front of the ##, see suggestion.

Thank you that's brilliant

[martrapp]

Sorry for the nitpicking. The main reason for the suggested changes are scripts like this:

<script type="module" data-astro-reload>...</script>

which are module scripts (= deferred loading) but not bundled (instead inlined) and can be re-executed without problems.

So much for: "scripts are run in sequential order"

<script type="module">
  console.log("i log later");
</script>

<script is:inline>
  console.log("i log first");
</script>

(both are inline even though it is only explicit for the second one)

[OliverSpeir]

I appreciate the nitpick especially when its as easy as hitting commit suggestions haha and I really do want this section of the docs to be as clear as possible because this is a huge part of the view tranistions experience

haha at least running a module will run when its meant to (deffered) but yeah the whole any attribute creates is:inline is weird

[OliverSpeir]

@sarah11918 Could you take another look at this when you get a chance?

[sarah11918]

Yup! These 4.5 docs are on deck for this week! 🙌

[yanthomasdev]

Thanks @OliverSpeir, just a few nits suggestions, after this it should be good to go 🙌

[OliverSpeir]

Thank you! Great catch

[sarah11918]

OK! Thanks for waiting patiently @OliverSpeir ! I went through this and think some restructuring of the content (both the original and yours) makes it even more helpful, so see whether my attempt makes sense here, and let me know what you think!

[OliverSpeir]

@sarah11918 brilliant suggestions thank you

[sarah11918]

Thanks, @OliverSpeir ! I really appreciate you adding the extra context over and above documenting astro-data-rerun!