Bug 275506

Summary: [view-transitions] Misplaced arrows after scrolling then transitioning on https://codepen.io/bramus/full/xxmozvN
Product: WebKit Reporter: Tim Nguyen (:ntim) <ntim>
Component: Layout and RenderingAssignee: Tim Nguyen (:ntim) <ntim>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://github.com/web-platform-tests/wpt/pull/46780
Bug Depends on:    
Bug Blocks: 259055    
Attachments:
Description Flags
Screen recording
none
Screen recording of slowed down animation
none
Test reduction
none
Even more reduced test case none

Tim Nguyen (:ntim)
Reported 2024-06-14 12:18:52 PDT
Created attachment 471680 [details] Screen recording ...
Attachments
Screen recording (9.06 MB, video/quicktime)
2024-06-14 12:18 PDT, Tim Nguyen (:ntim) 		no flags
Details
Screen recording of slowed down animation (19.30 MB, video/quicktime)
2024-06-15 08:24 PDT, Tim Nguyen (:ntim) 		no flags
Details
Test reduction (2.45 KB, text/html)
2024-06-15 08:53 PDT, Tim Nguyen (:ntim) 		no flags
Details
Even more reduced test case (588 bytes, text/html)
2024-06-15 09:31 PDT, Tim Nguyen (:ntim) 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2024-06-14 12:19:36 PDT
<rdar://problem/129866645>
Tim Nguyen (:ntim)
Comment 2 2024-06-15 08:24:25 PDT
Created attachment 471688 [details] Screen recording of slowed down animation
Tim Nguyen (:ntim)
Comment 3 2024-06-15 08:53:06 PDT
Created attachment 471689 [details] Test reduction A somewhat accurate reduction. I think the translation we apply of the scroll position (which is correct) applies in the wrong order. Rotation should apply, then translation. In this case, we see the translation being applied first, then that translation gets rotated.
Tim Nguyen (:ntim)
Comment 4 2024-06-15 09:31:26 PDT
Created attachment 471690 [details] Even more reduced test case
Tim Nguyen (:ntim)
Comment 5 2024-06-15 09:32:23 PDT
Scroll on the testcases to see the issue ^
Tim Nguyen (:ntim)
Comment 6 2024-06-16 22:47:03 PDT
Pull request: https://github.com/WebKit/WebKit/pull/29886
Tim Nguyen (:ntim)
Comment 7 2024-06-16 22:49:11 PDT
Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/46780
EWS
Comment 8 2024-06-17 11:55:46 PDT
Committed 280081@main (60a6419541bf): <https://commits.webkit.org/280081@main> Reviewed commits have been landed. Closing PR #29886 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.