Skip to content
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] view-transition-name: auto when matching id should namespace #11614

Open
vmpstr opened this issue Jan 30, 2025 · 3 comments
Labels
Agenda+ Async Resolution: Proposed Candidate for auto-resolve with stated time limit css-view-transitions-2 View Transitions; New feature requests

Comments

@vmpstr
Copy link
Member

vmpstr commented Jan 30, 2025

We have a view-transition-name: auto that can use the id attribute from the element as the view transition name. (https://www.w3.org/TR/css-view-transitions-2/#auto-vt-name)

I think it can be confusing if, for example, in the old state we match id=foo and on the other side that pairs with css view-transition-name: foo.

The proposal is to "namespace" id matching. Basically id=foo would only match with id=foo and not with view-transition-name: foo. Or, in other words, names generated by the id attribute can only match elements that have names that are also generated by the id attribute.

@nt1m @noamr fyi

@vmpstr vmpstr added Agenda+ css-view-transitions-2 View Transitions; New feature requests labels Jan 30, 2025
@nt1m
Copy link
Member

nt1m commented Jan 30, 2025

I think this makes sense

@noamr
Copy link
Collaborator

noamr commented Jan 30, 2025

I'm personally happy to resolve on this, perhaps asynchronously since WG meeting time is hard to get.

/cc @jakearchibald
I think this is less footgunny than the previous auto and is perhaps satisfactory.

@vmpstr vmpstr added the Async Resolution: Proposed Candidate for auto-resolve with stated time limit label Jan 30, 2025
@jakearchibald
Copy link
Contributor

I still think this is a bad feature with a bad name. But it was force-shipped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda+ Async Resolution: Proposed Candidate for auto-resolve with stated time limit css-view-transitions-2 View Transitions; New feature requests
Projects
None yet
Development

No branches or pull requests

4 participants