Skip to content

Adds a note about what internal state of elements can't be preserved by transition:persist#7292

Merged
martrapp merged 3 commits into
mainfrom
mt/persist
Mar 11, 2024
Merged

Adds a note about what internal state of elements can't be preserved by transition:persist#7292
martrapp merged 3 commits into
mainfrom
mt/persist

Conversation

@martrapp
Copy link
Copy Markdown
Member

Description (required)

A user on discord complained that the current text gives the impression that transition:persist would keep all states of an element, while he suffered from restarting animations.

He referred to this sentence, which he had taken out of context

the [...] internal state [...] will not be reset when navigating back and forth across pages

It might help to mention the known limitations.

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 11, 2024

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

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Mar 11, 2024 10:53pm
1 Ignored Deployment
Name Status Preview Updated (UTC)
docs-i18n ⬜️ Ignored (Inspect) Mar 11, 2024 10:53pm

Copy link
Copy Markdown
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Just some code formatting, otherwise, good to merge @martrapp ! Thank you for adding this clarification!

Comment thread src/content/docs/en/guides/view-transitions.mdx Outdated
@sarah11918 sarah11918 added improve or update documentation Enhance / update existing documentation (e.g. add example, improve description, update for changes) Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! labels Mar 11, 2024
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
@martrapp
Copy link
Copy Markdown
Member Author

Many thanks for the quick review and improvement!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improve or update documentation Enhance / update existing documentation (e.g. add example, improve description, update for changes) Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants