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

Issue 651735 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Non Regression: Unwanted vertical scrollbar is seen in camera app gallery.

Project Member Reported by bj00129...@techmahindra.com, Sep 30 2016

Issue description

Version: 55.0.2874.0/8848.0.0 dev-channel Paine,Glimmer,Butterfly
OS: Chrome

Precondition: Make sure you have 12-13 pictures in camera app gallery.

What steps will reproduce the problem?
(1)Sign in to user>>Go to camera app>>click on go to gallery icon and Observe unwanted vertical scrollbar (Please refer to the screenshots)

Note: 
1.Issue also seen on maximizing the window,unwanted scrollbar is seen and scrollbar disappears on clicking on it.
2. Issue not applicable to Linux and Windows.

Expected:No such scrollbar should be seen.
Actual: Instead unwanted scrollbar is seen in camera app gallery.

Raising this as Non-regression issue as it is also seen in 53.0.2785.103/8530.81.0 stable channel veyron_jerry.

@yuli: Please confirm the behavior.

Attaching screenshots for reference.
 
Scrollbar_minimize window.png
941 KB View Download
Scrollbar_maximize window.png
346 KB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 5 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/apps/camera/+/9a0e68aa6b6841bd29d32e86b16706e94246d1f7

commit 9a0e68aa6b6841bd29d32e86b16706e94246d1f7
Author: Yuli Huang <yuli@chromium.org>
Date: Mon Feb 05 06:02:47 2018

Fix scrollbar not updated for resizing the view.

BUG= chromium:651735 
TEST=Tested manually on elm.

Change-Id: I4e887f97e8fcf2f9aa5149053304190b4af67c6d
Reviewed-on: https://chromium-review.googlesource.com/896733
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>
Tested-by: yuli <yuli@chromium.org>

[modify] https://crrev.com/9a0e68aa6b6841bd29d32e86b16706e94246d1f7/src/js/views/browser.js
[modify] https://crrev.com/9a0e68aa6b6841bd29d32e86b16706e94246d1f7/src/js/views/album.js
[modify] https://crrev.com/9a0e68aa6b6841bd29d32e86b16706e94246d1f7/src/js/scrollbar.js

Comment 2 by y...@chromium.org, Feb 5 2018

Status: Fixed (was: Assigned)
Please verify it on the next camera app canary.
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 15

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9a0e68aa6b6841bd29d32e86b16706e94246d1f7

commit 9a0e68aa6b6841bd29d32e86b16706e94246d1f7
Author: Yuli Huang <yuli@chromium.org>
Date: Mon Feb 05 06:02:47 2018

Fix scrollbar not updated for resizing the view.

BUG= chromium:651735 
TEST=Tested manually on elm.

Change-Id: I4e887f97e8fcf2f9aa5149053304190b4af67c6d
Reviewed-on: https://chromium-review.googlesource.com/896733
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>
Tested-by: yuli <yuli@chromium.org>

[modify] https://crrev.com/9a0e68aa6b6841bd29d32e86b16706e94246d1f7/src/js/scrollbar.js
[modify] https://crrev.com/9a0e68aa6b6841bd29d32e86b16706e94246d1f7/src/js/views/album.js
[modify] https://crrev.com/9a0e68aa6b6841bd29d32e86b16706e94246d1f7/src/js/views/browser.js

Project Member

Comment 4 by bugdroid, Today (9 hours ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b4a50c8a00f311645635b4eb22053cbaf6747286

commit b4a50c8a00f311645635b4eb22053cbaf6747286
Author: Yuli Huang <yuli@chromium.org>
Date: Mon Feb 05 06:02:47 2018

Fix scrollbar not updated for resizing the view.

BUG= chromium:651735 
TEST=Tested manually on elm.

Change-Id: I4e887f97e8fcf2f9aa5149053304190b4af67c6d
Reviewed-on: https://chromium-review.googlesource.com/896733
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>
Tested-by: yuli <yuli@chromium.org>


Sign in to add a comment