Bug 152667 - Scroll snapping doesn't work in RTL scrollable content
Summary: Scroll snapping doesn't work in RTL scrollable content
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Scrolling (show other bugs)
Version: Safari 9
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 218115
  Show dependency treegraph
 
Reported: 2016-01-03 11:38 PST by Simon Fraser (smfr)
Modified: 2021-01-14 03:38 PST (History)
3 users (show)

See Also:


Attachments
Testcase (852 bytes, text/html)
2016-01-03 11:38 PST, Simon Fraser (smfr)
no flags Details
Updated test case (804 bytes, text/html)
2021-01-14 03:38 PST, Martin Robinson
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2016-01-03 11:38:01 PST
Created attachment 268146 [details]
Testcase

Scroll snapping is broken in RTL scrollable content. Test case attached.

We need to figure out of snap points are offsets (0 -> max) or positions (min -> max, possibly negative).
Comment 1 Simon Fraser (smfr) 2016-01-03 11:38:18 PST
appendChildSnapOffsets() filters out negative offsets.
Comment 2 Simon Fraser (smfr) 2020-10-20 15:24:33 PDT
I think I fixed this.
Comment 3 Martin Robinson 2021-01-14 03:38:17 PST
Created attachment 417606 [details]
Updated test case

I've updated the test case to follow the modern version of the specification and I can confirm that this is working as expected. I'm going to close this bug for now and then we can open new bugs if we find more specific issues with RTL content.