RESOLVED FIXED 273031
[Regression: Safari 17.2+] Scrollable elements with `pointer-events: none` prevent scroll events to pass through underlying scrollable sibling elements
https://bugs.webkit.org/show_bug.cgi?id=273031
Summary [Regression: Safari 17.2+] Scrollable elements with `pointer-events: none` pr...
Hernán Sartorio
Reported 2024-04-20 09:08:58 PDT
If I have a scrollable element A with `pointer-events: none` overlaid on top of another scrollable sibling element B with `pointer-events: auto`, I’d expect scrolling with the pointer over B would scroll element A but it does not. Tested in Safari 17.4.1 (macOS 14.4.1) and Safari Technology Preview Release 192. It does work as expected in Chrome and Firefox. Also in Safari 17.1 on macOS Ventura. Here’s an example reproducing it: https://codesandbox.io/p/devbox/safari-bug-kyzvjt
Attachments
Standalone test case (1.76 KB, text/html)
2024-04-24 17:25 PDT, Simon Fraser (smfr)
no flags
Radar WebKit Bug Importer
Comment 1 2024-04-22 09:39:58 PDT
Karl Dubost
Comment 2 2024-04-23 01:39:28 PDT
Thanks for the report and test case Hernan To better understand the steps to reproduce 1. Load https://codesandbox.io/p/devbox/safari-bug-kyzvjt?file=%2Findex.html 2. Over the cursor on the left area 3. Scroll (A) 4. Move the cursor on the middle area 5. Scroll (B) 6. Move the cursor the right area 7. Scroll (C) Check if Underlaid Content is being scrolled. Try this with the three browsers (latest versions): A B C Safari STP...... OK x x Firefox Nightly. OK OK x Chrome Canary... OK OK x
Ahmad Saleem
Comment 3 2024-04-23 06:11:00 PDT
Adding 'Regression' in title assuming from comment 0 that it was working as expected in Safari 17.1
Simon Fraser (smfr)
Comment 4 2024-04-24 17:25:01 PDT
Created attachment 471120 [details] Standalone test case
Simon Fraser (smfr)
Comment 5 2024-04-24 17:43:52 PDT
In RemoteScrollingTreeMac::scrollingNodeForPoint() I think the `isScrolledBy(*this, nodeID, frontmostLayer)` logic is wrong.
Matthieu Dubet
Comment 6 2024-04-25 17:07:33 PDT
EWS
Comment 7 2024-04-27 17:20:18 PDT
Committed 278084@main (34e7db744efe): <https://commits.webkit.org/278084@main> Reviewed commits have been landed. Closing PR #27779 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.