Bug 256995

Summary: horizontal scrollbars unusable with rtl element
Product: WebKit Reporter: Luke Warlow <lwarlow>
Component: ScrollingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: ahmad.saleem792, karlcow, lwarlow, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari 16   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
rendering in safari, firefox, chrome none

Description Luke Warlow 2023-05-18 16:22:04 PDT
data:text/html,<div class="container"> <div class="content">thin</div> </div> <style> .container { overflow-x: auto; height: 200px; width: 200px; direction: rtl; background: lightsalmon; } .content { width: 300px; } </style>

If you copy the above data url and open it in Safari on macOS (with non-overlay scrollbars visible), and try to scroll the content you wont be able to.

This works as expected in Chrome and Firefox.
Comment 1 Karl Dubost 2023-05-18 20:32:24 PDT
Created attachment 466412 [details]
rendering in safari, firefox, chrome

I could scroll horizontally with the trackpad on a MacBook.

Tested on macOS 13.4
---
Safari Technology Preview  170           18616.1.14.5
Firefox Nightly            115.0a1       11523.5.10
Google Chrome Canary       115.0.5762.0  5762.0
Comment 2 Luke Warlow 2023-05-19 09:39:02 PDT
Apologies to clarify I specifically meant I couldn't drag the scrollbar to scroll I could scroll using other mechanisms.

I'd have to double check but I also don't believe the scrollbar hover effect worked which makes me think it's an issue with the hit testing code.
Comment 3 Radar WebKit Bug Importer 2023-05-25 16:23:29 PDT
<rdar://problem/109858866>
Comment 4 Ahmad Saleem 2023-06-15 12:37:56 PDT
Most probably similar to 250207?

Or bug 249187 (if using mouse - but not horizontal case).
Comment 5 Ahmad Saleem 2023-06-15 12:40:07 PDT
(In reply to Ahmad Saleem from comment #4)
> Most probably similar to 250207?
> 
> Or bug 249187 (if using mouse - but not horizontal case).

You can see my fail PR attempt for first case here: https://github.com/WebKit/WebKit/pull/8355
Comment 6 Simon Fraser (smfr) 2023-12-21 17:13:04 PST
<rdar://104944522>
Comment 7 Simon Fraser (smfr) 2023-12-21 17:15:06 PST
Pull request: https://github.com/WebKit/WebKit/pull/22189
Comment 8 EWS 2023-12-22 17:08:41 PST
Committed 272466@main (d7611e7b6b1a): <https://commits.webkit.org/272466@main>

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