Skip to content

Conversation

@robertoms99
Copy link
Contributor

@robertoms99 robertoms99 commented Apr 8, 2025

Changes

Avoiding preserve data-astro-* attrs other than data-astro-transition and data-astro-transition-fallback when using swapRootAttributes #135577

Fixes #13577

Testing

Running packages/astro/e2e/view-transitions.test.js

Docs

It' s a patch
martrapp: bug fix, n/a

@changeset-bot
Copy link

changeset-bot bot commented Apr 8, 2025

🦋 Changeset detected

Latest commit: 7e5757f

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Apr 8, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 8, 2025

CodSpeed Performance Report

Merging #13587 will not alter performance

Comparing robertoms99:main (7e5757f) with main (5a0563d)

Summary

✅ 6 untouched benchmarks

Copy link
Member

@martrapp martrapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @robertoms99 thank you so much for the PR!
Looks very good and I also appreciate the improved naming.

I only suggest the changeset to be a bit more in line with what we typically write ;-)

@martrapp martrapp merged commit a0774b3 into withastro:main Apr 9, 2025
27 of 28 checks passed
@astrobot-houston astrobot-houston mentioned this pull request Apr 9, 2025
@martrapp
Copy link
Member

martrapp commented Apr 9, 2025

Really appreciate you spotting that and fixing it! Thanks again!

openscript pushed a commit to openscript/astro that referenced this pull request Sep 12, 2025
* fix: solving withastro#13577

* Update .changeset/yellow-dogs-say.md

Co-authored-by: Martin Trapp <[email protected]>

---------

Co-authored-by: Martin Trapp <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Not swapping data-astro-* attributes in client routing

2 participants