152667 2016-01-03 11:38:01 -0800 Scroll snapping doesn't work in RTL scrollable content 2021-01-14 03:38:31 -0800 1 1 1 Unclassified WebKit Scrolling Safari 9 Unspecified Unspecified RESOLVED WORKSFORME P2 Normal --- 218115 1 simon.fraser webkit-unassigned mrobinson simon.fraser wenson_hsieh oldest_to_newest 1152137 0 268146 simon.fraser 2016-01-03 11:38:01 -0800 Created attachment 268146 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). 1152138 1 simon.fraser 2016-01-03 11:38:18 -0800 appendChildSnapOffsets() filters out negative offsets. 1699961 2 simon.fraser 2020-10-20 15:24:33 -0700 I think I fixed this. 1719567 3 417606 mrobinson 2021-01-14 03:38:17 -0800 Created attachment 417606 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. 268146 2016-01-03 11:38:01 -0800 2021-01-14 03:38:17 -0800 Testcase main-frame-elements-snap-rtl.html text/html 852 simon.fraser CjwhRE9DVFlQRSBodG1sPgoKPGh0bWwgZGlyPSJydGwiPgo8aGVhZD4KICAgIDxzdHlsZT4KICAg ICAgICBib2R5IHsKICAgICAgICAgICAgbWFyZ2luOiAwOwogICAgICAgICAgICAtd2Via2l0LXNj cm9sbC1zbmFwLXR5cGU6IG1hbmRhdG9yeTsKICAgICAgICAgICAgLXdlYmtpdC1zY3JvbGwtc25h cC1wb2ludHMteDogZWxlbWVudHM7CiAgICAgICAgfQogICAgICAgIAogICAgICAgIC5jb250YWlu ZXIgewogICAgICAgICAgICB3aWR0aDogMjQwMHB4OwogICAgICAgIH0KICAgICAgIAogICAgICAg IC5jb250YWluZXIgPiBkaXYgewogICAgICAgICAgICBtYXJnaW46IDA7CiAgICAgICAgICAgIHBh ZGRpbmc6IDA7CiAgICAgICAgICAgIHdpZHRoOiA0MDBweDsKICAgICAgICAgICAgaGVpZ2h0OiA0 MDBweDsKICAgICAgICAgICAgZGlzcGxheTogaW5saW5lLWJsb2NrOwogICAgICAgICAgICAtd2Vi a2l0LXNjcm9sbC1zbmFwLWNvb3JkaW5hdGU6IDBweCAwcHg7CiAgICAgICAgfQogICAgPC9zdHls ZT4KPC9oZWFkPgo8Ym9keT4KICAgIDxkaXYgY2xhc3M9ImNvbnRhaW5lciI+CiAgICAgICAgPGRp diBzdHlsZT0nYmFja2dyb3VuZC1jb2xvcjogcmVkJz48L2Rpdj48ZGl2IHN0eWxlPSdiYWNrZ3Jv dW5kLWNvbG9yOiBncmVlbic+PC9kaXY+PGRpdiBzdHlsZT0nYmFja2dyb3VuZC1jb2xvcjogYmx1 ZSc+PC9kaXY+PGRpdiBzdHlsZT0nYmFja2dyb3VuZC1jb2xvcjogYXF1YSc+PC9kaXY+PGRpdiBz dHlsZT0nYmFja2dyb3VuZC1jb2xvcjogeWVsbG93Jz48L2Rpdj48ZGl2IHN0eWxlPSdiYWNrZ3Jv dW5kLWNvbG9yOiBmdWNoc2lhJz48L2Rpdj4KICAgIDwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4K 417606 2021-01-14 03:38:17 -0800 2021-01-14 03:38:17 -0800 Updated test case attachment.htm text/html 804 mrobinson CjwhRE9DVFlQRSBodG1sPgoKPGh0bWwgZGlyPSJydGwiPgo8aGVhZD4KICAgIDxzdHlsZT4KICAg ICAgICBodG1sIHsKICAgICAgICAgICAgc2Nyb2xsLXNuYXAtdHlwZTogeCBtYW5kYXRvcnk7CiAg ICAgICAgfQogICAgICAgIGJvZHkgewogICAgICAgICAgICBtYXJnaW46IDA7CiAgICAgICAgfQog ICAgICAgIAogICAgICAgIC5jb250YWluZXIgewogICAgICAgICAgICB3aWR0aDogMjQwMHB4Owog ICAgICAgIH0KICAgICAgIAogICAgICAgIC5jb250YWluZXIgPiBkaXYgewogICAgICAgICAgICBt YXJnaW46IDA7CiAgICAgICAgICAgIHBhZGRpbmc6IDA7CiAgICAgICAgICAgIHdpZHRoOiA0MDBw eDsKICAgICAgICAgICAgaGVpZ2h0OiA0MDBweDsKICAgICAgICAgICAgZGlzcGxheTogaW5saW5l LWJsb2NrOwogICAgICAgICAgICBzY3JvbGwtc25hcC1hbGlnbjogc3RhcnQ7CiAgICAgICAgfQog ICAgPC9zdHlsZT4KPC9oZWFkPgo8Ym9keT4KICAgIDxkaXYgY2xhc3M9ImNvbnRhaW5lciI+CiAg ICAgICAgPGRpdiBzdHlsZT0nYmFja2dyb3VuZC1jb2xvcjogcmVkJz48L2Rpdj48ZGl2IHN0eWxl PSdiYWNrZ3JvdW5kLWNvbG9yOiBncmVlbic+PC9kaXY+PGRpdiBzdHlsZT0nYmFja2dyb3VuZC1j b2xvcjogYmx1ZSc+PC9kaXY+PGRpdiBzdHlsZT0nYmFja2dyb3VuZC1jb2xvcjogYXF1YSc+PC9k aXY+PGRpdiBzdHlsZT0nYmFja2dyb3VuZC1jb2xvcjogeWVsbG93Jz48L2Rpdj48ZGl2IHN0eWxl PSdiYWNrZ3JvdW5kLWNvbG9yOiBmdWNoc2lhJz48L2Rpdj4KICAgIDwvZGl2Pgo8L2JvZHk+Cjwv aHRtbD4K