You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[css-view-transitions-1] What is the size of the group pseudo for the root element? #10177
Chrome seems to be setting the size of their ::view-transition-group element to that of the snapshot containing block. That seems wrong, but visually what you’d expect to happen.
Capture the image algorithm has a special case for sizing the image when capturing the root, I suspect Update pseudo-element styles should have the equivalent wording for setting the width/height style properties.
That seems wrong, but visually what you’d expect to happen.
Just to confirm, the error is that we're using the border-box for the document element here. Instead it should be the snapshot containing block size similar to when we do this for the old DOM here.
Just to confirm, the error is that we're using the border-box for the document element here. Instead it should be the snapshot containing block size similar to when we do this for the old DOM here.
Activity
nt1m commentedon Apr 5, 2024
@noamr @khushalsagar @vmpstr
noamr commentedon Apr 5, 2024
Sounds reasonable to me, will wait for Vlad/Khush to confirm. Happy to make the edit.
khushalsagar commentedon Apr 8, 2024
Just to confirm, the error is that we're using the border-box for the document element here. Instead it should be the snapshot containing block size similar to when we do this for the old DOM here.
SGTM. Thanks for spotting this!
[css-view-transitions-1] Use snapshot containing block for new root
mattwoodrow commentedon Apr 8, 2024
Yes, exactly. Thank you!
[css-view-transitions-1] Use snapshot containing block for new root (#…