-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 experimental ViewTransition
interface
#3598
Conversation
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.
LGTM!
Could you add an entry to the changelog?
Added. |
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.
Thanks!
Just waiting for CI.
@daxpedda I wanted to ask an additional question, when can it be moved into the |
We don't have any official guidelines for this. Basically when we think that the API isn't going to change anymore. It's a candidate recommendation, so I'm assuming this isn't going to change anymore, but it's only implemented by Chromium and has very low usage across the web. Considering how close this is to recommendation I would prefer to wait unless there is a strong incentive otherwise. |
Alright. If you're wondering, the reason why I added this is because someone requested this as a feature (as in, a framework API around it) in Dioxus, which is an SPA framework and thus would benefit quite a bit from this. |
This adds the experimental
ViewTransition
interface. See MDN docs and W3C specs. I hope experimental APIs are allowed.I followed the contributing guidelines, but let me know if I did something wrong.