New issue
Advanced search Search tips

Issue 679309 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 718515
Owner: ----
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Image jumps around unpredictably while scrolling

Project Member Reported by skyos...@chromium.org, Jan 9 2017

Issue description

Chrome Version: 57.0.2970.0
OS: Android

What steps will reproduce the problem?
(1) Navigate to https://i.redd.it/cvp2j8njdn5y.jpg
(2) Zoom in.
(3) Scroll around the image.

What is the expected result?
Image scroll predictably.

What happens instead?
When you lift your finger, the image often jumps around to a different position. This seems to be caused by the omnibox getting shown and hidden.

 
Status: Untriaged (was: Available)

Comment 2 by bokan@chromium.org, Jan 12 2017

Cc: bokan@chromium.org
Labels: Hotlist-Input-Dev Hotlist-GoodFirstBug OS-Android
Status: Available (was: Untriaged)
This is happening because we resize the viewport when the URL bar is hidden/shown but we try to manually keep the image centered when the viewport size changes in ImageDocument::windowSizeChanged. Starting in 56, we no longer resize the layout due to the URL bar to prevent these kinds of "jump around" issues but we're manually adjusting the values here in C++.

Presumably there's some reason we did that but we should probably change this to use CSS and percentage values to layout.
Project Member

Comment 3 by sheriffbot@chromium.org, Feb 15 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by bokan@chromium.org, Feb 15 2018

Mergedinto: 718515
Status: Duplicate (was: Untriaged)
This was fixed in  issue 718515 . 

Sign in to add a comment