Bug 259684 – Overflow element non-interactive after removing inert attribute

• Status: RESOLVED
• Resolution: FIXED
• Component: UI Events

[Dan Palmer]

Created attachment 467165 [details]
HTML file with usecase

Overview: Applying inert to an element with overflow:scroll, and then removing the inert attribute, still keeps it non-interactive.

Steps to Reproduce:

1) Create an element with overflow:scroll, height:200px and inert.

2) Add some paragraphs of text within the element.

3) Create a button that when clicked will remove the inert attribute from the element.

4) Load the page.

Actual Results:

No scrolling on the element. Click the button to remove inert. Element still does not scroll.

Expected Results: 

After clicking the button to remove inert, the element should scroll.

Build Date & Hardware:

iPhone Useragent: Mozilla/5.0 (iPhone; CPU iPhone OS 16_5_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.5.2 Mobile/15E148 Safari/604.1

Additional Information: 

If you include a second overflow:scroll element below the first and scroll it after you have removed inert from the first, scrolling on the first becomes possible again.

[Radar WebKit Bug Importer]

<rdar://problem/113239461>

[Tim Nguyen (:ntim)]

Created attachment 467233 [details]
Testcase using pointer-events

[Tim Nguyen (:ntim)]

This bug is specific to inert. Probably just needs a special check here: https://searchfox.org/wubkat/rev/33c49ff64449389431294ba0ace6f7d0ad6306b7/Source/WebCore/rendering/style/RenderStyle.cpp#1350-1351

[Simon Fraser (smfr)]

I think we have some UIScrollView-specific code that deals with interactivity too.

[Tim Nguyen (:ntim)]

(In reply to Simon Fraser (smfr) from comment #4)
> I think we have some UIScrollView-specific code that deals with
> interactivity too.

Given this doesn't reproduce with pointer-events, it probably just needs the same treatment as pointerEvents here.

[Tim Nguyen (:ntim)]

Pull request: https://github.com/WebKit/WebKit/pull/16800

[EWS]

Committed 267030@main (cd37141f0b59): <https://commits.webkit.org/267030@main>

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