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

Issue 651308 link

Starred by 0 users
Status: Verified
Owner:
dpa...@chromium.org
Closed: Oct 2016
Cc:
nyerramilli@chromium.org
dbeam@chromium.org
█ tsergeant@chromium.org
michae...@chromium.org
█ ranjitkan@chromium.org
hcarmona@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

In chrome://settings, able to scroll the page using mouse wheel even when dropdown menu is open.

Reported by vvishwak...@etouch.net, Sep 29 2016 
Version: 55.0.2874.4 (Official Build) canary 6f65ea7f427cb2712e2222e9248ae4399a619735-refs/branch-heads/2874@{#5} (32/64-bit)
OS: Windows (7,8,10), Mac (10.10.5, 10.11.5), Linux (14.04 LTS)

What steps will reproduce the problem?
1) Launch chrome, go to chrome://md-settings, scroll down and click on Advanced button.
2) in Language section, click on dropdown on right side and then click on iron-icon for English(United States).
3) Now scroll the page up/down using mouse wheel and observe.

Able to scroll the page even when dropdown menu is open. 

Should be not able to scroll the page with dropdown menu open.

This is a Regression issue broken in M-55, will soon update other info
Manual bisect:
Good build: 55.0.2841.0
Bad build: 55.0.2842.0
languages_actual.mov
4.7 MB Download
language_expected.mov
2.7 MB Download

Comment 1 by msrchandra@chromium.org, Sep 29 2016

Labels: -hasbisect -Type-Bug-Regression hasbisect-per-revision Type-Bug
Owner: michae...@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 55.0.2841.0 (Revision: 414854).
Bad build: 55.0.2842.0 (Revision: 414942).

You are probably looking for a change made after 414890 (known good), but no later than 414891 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/16e132a1b213a07254d25b50a42ba9e9304d3f75..b44ce49d6fa9df4aa44b14c7d5df3416ff1392ea

@michaelpg -- Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.
Thank You.

Comment 2 by michae...@chromium.org, Sep 29 2016

Cc: dbeam@chromium.org michae...@chromium.org tsergeant@chromium.org hcarmona@chromium.org
Labels: -M-55 Proj-MaterialDesign-WebUI OS-Chrome
Owner: ----
Status: Available (was: Assigned)
Summary: In chrome://settings, able to scroll the page using mouse wheel even when dropdown menu is open. (was: Regression: In chrome://settings, able to scroll the page using mouse wheel even when dropdown menu is open.)
Autofill and Passwords have the same problem, due to also using cr-shared-menu.

History solves this by closing the menu when the container receives a scroll event.

It would be nice if cr-shared-menu itself could listen for scroll events, but it would have to "know" all of the different targets that could scroll, which seems unfeasible.

Comment 3 by dbeam@chromium.org, Sep 29 2016

Owner: dpa...@chromium.org
Status: Assigned (was: Available)
dpapad@ is reimagining how shared action menus would work

our current plan is to reparent each menu actually into the DOM where it's requested, which adds potential clipping issues but solves scroll issues.

Comment 4 by dpa...@chromium.org, Oct 19 2016

Status: Fixed (was: Assigned)
Fixed by https://codereview.chromium.org/2426183002 and https://codereview.chromium.org/2428733002.

Comment 5 by sdantul...@chromium.org, Oct 24 2016

Status: Verified (was: Fixed)
Verified on ChromeOS 8929.0.0, 56.0.2899.0
►

Sign in to add a comment