Issue metadata
Sign in to add a comment
|
Regression: [MD Settings] Weird behavior of page scrolling is observed.
Reported by
dchau...@etouch.net,
Jun 30 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 53.0.2784.0 (Official Build)642ae4ba1dc7b04a1d480d733cdced28de720e00-refs/heads/master@{#403038} 32/64-bit. OS: Windows(7,8,10), Mac (10.10.5)(10.11.4), Linux(Ubuntu 14.04 LTS). What steps will reproduce the problem? 1. Launch chrome, go chrome://md-settings page and then click on 'Advanced' button. 2. Go to 'Language' section, Click on iron-icon and then click on 'Manage language' button. 3. Now, click on back navigation arrow button on the overlay and observe the MD Settings page. Unnecessary page is scrolled up after clicking back navigation arrow button. Page should not scroll up (it should stay on 'Language' section) after clicking on back navigation arrow button. This is a regression issue, broken in M-53 series, below is bisect info. Good build: 53.0.2761.0 Bad build: 53.0.2763.0 Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/42947e24de37b28e80c8456fdf36460c5296db63..f97dc9cd33155967950da6b4ed68da34744fc6e7?pretty=fuller&n=100 Suspecting: r398434 Note: This issue is reproducible for all the overlay on chrome://md-settings page. Kindly review the attached screen-cast for reference.
,
Jul 22 2016
@tommycli, this is that bug about saving the scroll position in the routing history. So that using the browser back button will scroll to the prior position. It makes sense that the routing refactoring should happen before addressing this.
,
Jul 22 2016
#2: This is actually a bug in the animation logic from when we combined Basic and Advanced. The scroll position gets offset when the section expands and collapses. crrev.com/2106013002/ fixes this. Issue 537359 is more about saving scroll position. The transitions should take care of restoring scroll position when you go back once, but if you go back multiple history entries repeatedly we have no context for scroll position.
,
Jul 22 2016
,
Aug 2 2016
Issue 632381 has been merged into this issue.
,
Sep 7 2016
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by sheriffbot@chromium.org
, Jul 1 2016