Bug 242224 - The "Around the World" carousel is broken on https://focal-theme-carbon.myshopify.com/
Summary: The "Around the World" carousel is broken on https://focal-theme-carbon.mysho...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Scrolling (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-06-30 21:03 PDT by Simon Fraser (smfr)
Modified: 2022-08-17 01:15 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2022-06-30 21:03:45 PDT
Load https://focal-theme-carbon.myshopify.com/ and scroll down to the "Around the World" section. Click the prev/next arrows, and note how it fails to scroll to the right place.
Comment 1 Radar WebKit Bug Importer 2022-07-05 10:59:33 PDT
<rdar://problem/96453957>
Comment 2 Simon Fraser (smfr) 2022-08-16 13:31:40 PDT
During the scroll animation, layout runs and calls `FrameView::removeScrollableArea()` so we stop animating it.
Comment 3 Simon Fraser (smfr) 2022-08-16 14:04:46 PDT
Pull request: https://github.com/WebKit/WebKit/pull/3377
Comment 4 EWS 2022-08-16 17:08:03 PDT
Committed 253493@main (ca608f18f089): <https://commits.webkit.org/253493@main>

Reviewed commits have been landed. Closing PR #3377 and removing active labels.
Comment 5 Ryosuke Niwa 2022-08-17 00:52:38 PDT
This patch broke iOS builds.
Comment 6 Ryosuke Niwa 2022-08-17 01:05:59 PDT
Re-opening for pull request https://github.com/WebKit/WebKit/pull/3396
Comment 7 Ryosuke Niwa 2022-08-17 01:08:28 PDT
Pull request: https://github.com/WebKit/WebKit/pull/3397
Comment 8 EWS 2022-08-17 01:15:29 PDT
Committed 253511@main (711178d9ca19): <https://commits.webkit.org/253511@main>

Reviewed commits have been landed. Closing PR #3397 and removing active labels.