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

Issue 728111 link

Starred by 1 user
Status: Fixed
Owner:
steve...@chromium.org
Closed: Jun 2017
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
hcarmona@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary hand pointer is seen on text of chrome://settings/searchEngines page.

Reported by db...@etouch.net, May 31 2017 
Chrome Version: 61.0.3116.0 64a9be37ccf1e29e9e9cc302da2297f2d3661b5d-refs/heads/master@{#475721}
OS: Windows (7,8,10),Linux (14.04 LTS),Mac OS X(10.11.6,10.12.3)

What steps will reproduce the problem?
(1) Launch chrome, navigate to chrome://settings/searchEngines page
(2) Hover mouse under default search engines and observe pointer. 

Actual: Unnecessary hand pointer is seen on text which is not even clickable.

Expected: Hand pointer should not seen if its not clicakble.

This is a regression issue, broken in 'M-60', will soon update the other info:

Good Build:60.0.3100.0
Bad Build: 60.0.3102.0
Actual_Cursor.mp4
311 KB View Download
Expected_Cursor.mp4
201 KB View Download

Comment 1 by rbasuvula@chromium.org, May 31 2017

Cc: rbasuvula@chromium.org
Labels: hasbisect-per-revision
Owner: hcarmona@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build:60.0.3100.0 (Revision:471639).
Bad build:60.0.3102.0 (Revision:472262).

You are probably looking for a change made after 471967 (known good), but no later than 471968 (first known bad).

CHANGE-LOG URL:
---------------
https://chromium.googlesource.com/chromium/src/+log/fd1fe22c153c500c60ffd85cb32eb6576e8ea1ad..1af97c4930cb92df4112db82a79ddacd0041379c

From the CL above, assigning the issue to the concern owner

@hcarmona: 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.

Review-Url:  https://codereview.chromium.org/2855273002
Note :Able to reproduce the issue in Win 10.0,Ubuntu 14.04 & Mac 10.12.3 and Able to reproduce in latest Canary #61.0.3116.0

Comment 2 by hcarmona@chromium.org, May 31 2017

Cc: hcarmona@chromium.org
Owner: steve...@chromium.org
Steven, PTAL

The cursor seems to come from https://cs.chromium.org/chromium/src/ui/webui/resources/cr_elements/shared_style_css.html?rcl=a62c0d824200e6e00806c7be02c117ad7b85dbea&l=104

This might not need to be applied to all iron lists.

Comment 3 by steve...@chromium.org, Jun 2 2017

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 5 2017 

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

commit 20cb815a45b037b022d89f4ce1e105eca89ab69c
Author: stevenjb <stevenjb@chromium.org>
Date: Mon Jun 05 17:50:51 2017

Settings: Do not apply cr-actionable to scrollable items

The only scrollable container with actionable items that are not
already identified are bluetooth devices, so just fix those.

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

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

[modify] https://crrev.com/20cb815a45b037b022d89f4ce1e105eca89ab69c/chrome/browser/resources/settings/bluetooth_page/bluetooth_subpage.html
[modify] https://crrev.com/20cb815a45b037b022d89f4ce1e105eca89ab69c/ui/webui/resources/cr_elements/shared_style_css.html

Comment 5 by steve...@chromium.org, Jun 5 2017

Status: Fixed (was: Started)
►

Sign in to add a comment