Overlay scrollbars appear abruptly |
|||||||
Issue descriptionChrome Version: 59.0.3065.0 OS: Chrome What steps will reproduce the problem? (1) Let overlay scrollbars fade out (2) Move mouse to within 30px of scrollbars What is the expected result? Scrollbars appear, expanded What happens instead? Scrollbars do not appear. The scrollbars don't appear until the mouse is directly on top of them. Spec: - When the scrollbars are hidden, they should become both visible and expanded when the mouse is within 30px of the side of the screen. - When the scrollbars are visible but narrow, they should become expanded when the mouse is within 30px of the side of the screen. Please use labels and text to provide additional information. For graphics-related bugs, please copy/paste the contents of the about:gpu page at the end of this report.
,
Apr 8 2017
,
Apr 10 2017
I can repro and found the issue I think: https://cs.chromium.org/chromium/src/cc/input/scrollbar_animation_controller.cc?l=328 This line should be (kMouseMoveDistanceToTriggerShow + scrollbar->ThumbThickness()))
,
Apr 10 2017
,
Apr 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/07077943755bf2cf166a2fe8c2aaab6fd46142b6 commit 07077943755bf2cf166a2fe8c2aaab6fd46142b6 Author: chaopeng <chaopeng@chromium.org> Date: Wed Apr 12 20:06:24 2017 Change the distance trigger scrollbar show We used to calculate the distance from edge to trigger scrollbar show. Now change to use the distance from scrollbar to trigger scrollbar show. BUG= 709700 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel Review-Url: https://codereview.chromium.org/2813643002 Cr-Commit-Position: refs/heads/master@{#464125} [modify] https://crrev.com/07077943755bf2cf166a2fe8c2aaab6fd46142b6/cc/input/scrollbar_animation_controller.cc [modify] https://crrev.com/07077943755bf2cf166a2fe8c2aaab6fd46142b6/cc/input/scrollbar_animation_controller_unittest.cc
,
Apr 13 2017
,
Apr 13 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-59; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-59 label, otherwise remove Merge-TBD label. Thanks.
,
Apr 13 2017
Re#8, I thought M-59 still hasn't branched as of today. Was the above comment a mistake?
,
Apr 20 2017
verified on build 9460.4.0
,
Jun 2 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by tbuck...@chromium.org
, Apr 8 2017