Regression: Unnecessary auto-scrolling of chrome://md-settings/reset page is seen in swanky device. |
||||||||||
Issue descriptionChrome Version: 58.0.3001.0/9254.0.0 dev-channel Swanky OS: Chrome What steps will reproduce the problem? (1)Navigate to chrome://md-settings page>> Open settings main menu and click on Advanced link (2)Now click on reset option and Observe Expected: No such auto scrolling of chrome://md-settings/reset page should be seen. Actual: Instead unnecessary auto-scrolling of chrome://md-settings/reset page is seen This is regression issue as it is working fine in 55.0.2883.82/8872.67.0 stable channel swanky. Issue is also seen in 56.0.2924.87/9000.82.0 beta channel swanky. Attaching screen-cast for reference. Note: 1. Issue is seen only in swanky device. 2. Issue not seen in Linux and also in wolf and glimmer with same build.
,
Feb 7 2017
,
Feb 8 2017
Michael, do you have an idea why this would only be happening on this device ('swanky device')? I'm wondering if it may be animation related or maybe the extra padding dynamically added at the end of the page.
,
Feb 8 2017
Steven, are you familiar with / have access to a 'swanky device'?
,
Feb 8 2017
It is not device specific. I can reproduce this on a Pixel (ToT only, not on 57) by setting the page zoom level to 90%. I was able to reproduce it at different screen resolutions (1440x956 or 1600x1062), but only with the page zoom set to 90%.
,
Feb 9 2017
,
Feb 9 2017
CL 2686863003
,
Feb 9 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/860ef963e7ecec486cce0690a1907581a9d1e581 commit 860ef963e7ecec486cce0690a1907581a9d1e581 Author: dschuyler <dschuyler@chromium.org> Date: Thu Feb 09 19:48:12 2017 [MD settings] prevent scrolling contents due to floating point noise This CL rounds up a floating point value to avoid a slow automatic scrolling in the md-settings. BUG= 688947 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2686863003 Cr-Commit-Position: refs/heads/master@{#449378} [modify] https://crrev.com/860ef963e7ecec486cce0690a1907581a9d1e581/chrome/browser/resources/settings/settings_main/settings_main.js
,
Feb 10 2017
dschuyler: for future reference, swanky is (apparently) the 2014 Toshiba Chromebook 2, and Googlers can find this info at go/crosnames. But yeah, this isn't device-specific or even Chrome OS-specific.
,
Feb 15 2017
,
Mar 22 2017
Verified Chrome OS version 58.0.3029.31/934.18.0 kevin
,
Jun 15 2017
Issue 653033 has been merged into this issue. |
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by omrilio@chromium.org
, Feb 7 2017Status: Assigned (was: Untriaged)