-
Notifications
You must be signed in to change notification settings - Fork 684
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
[css-view-transitions-2] MPA: What happens if there's a cross-origin redirect? #8684
Comments
Agreed. |
I don't think there is any disagreement here. Can we close this issue? It's in the explainer. |
As long as folks remember to put it in a spec |
Thinking about this again while working on a draft spec, disallowing cross-origin redirects might lead to strange behaviors:
Do we run transitions on the back/forward navigations, even though this would expose that there was a cross-origin redirect? Also, regarding "it seems bad to expose that there was no in-between page" - it's already exposed by calling Even without any new VT API, the developer can today:
|
Speaking with @annevk about this, one valid reason to disallow transitions on cross-origin redirects though is so that the user doesn't feel that things are "smoothed out", they should probably be aware somehow that there was a redirect, even if it's minor like a flash-of-white etc. If this is the reason to disallow VT on XO-redirect, it's pretty clear to spec, and wouldn't apply to history traversals as the redirect doesn't happen then and it's ok to keep them smooth. |
Closing, as this is in the draft spec. https://drafts.csswg.org/css-view-transitions-2/#document-setup-outbound-cross-document-view-transition |
Previous discussion: WICG/view-transitions#200
If you navigate from origin A to origin B, which redirects back to A, it seems bad to expose that there was no in-between page by allowing a transition to run or not.
So, if there's a cross-origin redirect, it feels like we should disallow transitions. We can lift the behaviour in future if we feel it's safe.
The text was updated successfully, but these errors were encountered: