New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 624754 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug-Regression

Blocking:
issue 589681



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 description

Chrome 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.
 
Actual behavior.mp4
1.5 MB View Download
Expected_behavior.mp4
799 KB View Download
Project Member

Comment 1 by sheriffbot@chromium.org, Jul 1 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: tommycli@chromium.org
@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.
Blocking: 589681
Cc: tommycli@chromium.org
Labels: -OS-Linux -OS-Windows -OS-Mac Proj-MaterialDesign-WebUI OS-All
Owner: michae...@chromium.org
Status: Started (was: Assigned)
#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.
Labels: Hotlist-MD-Settings-Navigation
 Issue 632381  has been merged into this issue.
Status: Fixed (was: Started)

Sign in to add a comment