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

Issue 645190 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Ripple effect appears on mouseup, not mousedown.

Reported by jleedev@gmail.com, Sep 8 2016

Issue description

Expect behavior, as in md-history:
- Open the sidebar and press the mouse down on an item. The ripple appears on mouse down.

Actual behavior in md-settings:
- Open the sidebar and press the mouse down on an item. The ripple doesn't appear until mouse up. This is barely visible, and is inconsistent.
 
Labels: -Pri-2 Hotlist-MD-Settings-Structure Pri-3
Owner: dschuyler@chromium.org
Status: Assigned (was: Unconfirmed)
We should have consistent ripple behavior. Though for MD Settings, we only show a ripple for the "Advanced" toggle since other items will cause the sidenav to close.
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 16 2016

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

commit b9adedb441e090b88388e0f61333163967bc0264
Author: dschuyler <dschuyler@chromium.org>
Date: Fri Sep 16 23:46:43 2016

[MD settings] ripple on mouse down in side nav

This CL changes the ripple to trigger on mouse down rather than mouse
up. This better matches other uses of paper-ripple.

BUG= 645190 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2341593002
Cr-Commit-Position: refs/heads/master@{#419331}

[modify] https://crrev.com/b9adedb441e090b88388e0f61333163967bc0264/chrome/browser/resources/settings/settings_menu/settings_menu.html
[modify] https://crrev.com/b9adedb441e090b88388e0f61333163967bc0264/chrome/browser/resources/settings/settings_menu/settings_menu.js

Status: Fixed (was: Started)

Sign in to add a comment