Not resetting scroll position when navigating to "About"
Reported by
jshan...@etouch.net,
Nov 22 2016
|
||||||
Issue descriptionChrome Version:56.0.2924.3 (Official Build) 66f73b3ae5a592d70e00602e87a783535ffe967e-refs/branch-heads/2924@{#54} OS: Windows (7,8,10), Mac (10.11.6, 10.12.1), Linux (14.04 LTS) Steps: 1. Launch Chrome and navigate to chrome://md-settings 2. Zoom more than 150% or 250% (using 'Ctrl' '+') and scroll down the page. 3. Now Click on 'Main menu' icon, select 'About chrome' and observe. Actual: 'About chrome' page is not navigated properly i.e it is seen auto scrolled down Expected: 'About chrome' page should be navigated properly. This is a regression issue broken in M-55 series, will soon update the bisect info. Good build: 55.0.2876.0 Bad build: 55.0.2878.0
,
Nov 23 2016
This bug is not correlated with scroll anchoring (repros with scroll anchoring disabled in about:flags), so it is not related to the change mentioned in #1. I am not sure it is a regression either, as I can repro in stable (54.0.2840.99), though not consistently. Assuming the "About chrome" link is triggering a client-side navigation I would expect the MD Settings javascript to reset the scroll position. @dbeam can you take a look or reroute as appropriate? Feel free to kick it back to me if you think Blink scrolling code is doing the wrong thing here.
,
Dec 2 2016
Just to to update the latest behavior of the bug, Issue is still observed on windows 10 chrome latest canaray M57-57.0.2938.0 dbeam@ Could you please let us know is there any recent update available on this issue? Thanks!
,
Dec 7 2016
Just to update the latest behavior of the bug, Issue is still observed on chrome latest Dev M57-57.0.2944.0 dbeam@ Could you please let us know is there any recent update available on this issue? Thanks!
,
Jan 7 2017
This occurs regardless of zoom level. Repro: 1) Change browser height so that About page scrolls 2) Go to chrome://md-settings and scroll partially 3) Change to About page via sidenav Seems like we just aren't resetting scroll when switching to the About page.
,
Jan 12 2017
,
Jan 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/52d38e7a5ae31b750d87205b072a324cf99926ef commit 52d38e7a5ae31b750d87205b072a324cf99926ef Author: tommycli <tommycli@chromium.org> Date: Wed Jan 18 01:03:36 2017 MD Settings: Scroll to top when navigating to About page. BUG= 667671 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2627403003 Cr-Commit-Position: refs/heads/master@{#444217} [modify] https://crrev.com/52d38e7a5ae31b750d87205b072a324cf99926ef/chrome/browser/resources/settings/settings_page/main_page_behavior.js [modify] https://crrev.com/52d38e7a5ae31b750d87205b072a324cf99926ef/chrome/test/data/webui/settings/basic_page_browsertest.js
,
Jan 18 2017
,
Jan 24 2017
Verified on Chrome OS 9202.1.0, 57.0.2987.6 |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by hdodda@chromium.org
, Nov 22 2016Labels: hasbisect-per-revision
Owner: skobes@chromium.org
Status: Assigned (was: Unconfirmed)