New issue
Advanced search Search tips

Issue 819769 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 497851



Sign in to add a comment

Implement scroll snapping for scrollbar dragging

Project Member Reported by sunyunjia@chromium.org, Mar 7 2018

Issue description

After user dragging the scrollbar, we should respect the snap points of the scrolled container if specified.

 
Blocking: 497851
Cc: majidvp@chromium.org
Summary: Implement Scroll Snap for scrollbar dragging (was: Snap after scrollbar dragging)
Summary: Implement scroll snapping for scrollbar dragging (was: Implement Scroll Snap for scrollbar dragging)
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a895624596bea9349c5ce3758f89122e306f16a5

commit a895624596bea9349c5ce3758f89122e306f16a5
Author: Sandra Sun <sunyunjia@chromium.org>
Date: Tue Mar 20 02:55:41 2018

Snap after scrollbar dragging

After user dragging the scrollbar thumb, we should snap to the closest
snap position of the container.

Bug:  819769 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I9d383d61a4f171d02c339064392216e0f4eae2ca
Reviewed-on: https://chromium-review.googlesource.com/953647
Commit-Queue: Sandra Sun <sunyunjia@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544271}
[add] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/LayoutTests/fast/scroll-behavior/snaps-after-scrollbar-dragging.html
[modify] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/Source/core/frame/LocalFrameView.cpp
[modify] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/Source/core/frame/LocalFrameView.h
[modify] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
[modify] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h
[modify] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/Source/platform/scroll/ScrollableArea.cpp
[modify] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/Source/platform/scroll/ScrollableArea.h
[modify] https://crrev.com/a895624596bea9349c5ce3758f89122e306f16a5/third_party/WebKit/Source/platform/scroll/Scrollbar.cpp

Status: Fixed (was: Assigned)

Sign in to add a comment