Regression: Unnecessary I beam pointer is seen for "Clear and reset" button at chrome://md-settings/siteSettings/cookies.
Reported by
jshan...@etouch.net,
Aug 9 2016
|
|||
Issue descriptionChrome Version: 54.0.2824.0 (Official Build) facabd3224aecbcab4bea9daadad31c67488d78c-refs/heads/master@{#410520} 32/64-bit. OS: Windows (7,8,8.1,10), Mac(10.10.5, 10.11.4), Linux (14.04 LTS). Steps: 1. Launch chrome and navigate to chrome://md-settings/siteSettings/cookies. 2. Set up an exception and then click on exception such that it navigate to chrome://md-settings/siteSettings/cookies/details. 3. Hover mouse on "Clear and reset" button and observe. Actual: Unnecessary I beam pointer is seen for "Clear and reset" button instead of hand pointer. Expected: I beam pointer should not be seen for "Clear and reset" button, hand pointer should be seen. This is a regression issue broken in M-54, below is bisect info. Good Build: 54.0.2819.0 Bad Build : 54.0.2820.0 Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/34ad8d1edc48ea257af035c270432475bac09755..5fcccaf7ad4b872c97c51bc04d364f0adefd3984?pretty=fuller&n=100 Suspecting: r409886 ? Please help to re-assign if your change is not the cause for this issue.
,
Aug 19 2016
FYI this effect also happens on the OnStartup URL list in the basic settings. The same fix will handle both cases.
,
Aug 22 2016
,
Aug 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/51071f55a9c5b147ec42660c1e27416098ec8cb8 commit 51071f55a9c5b147ec42660c1e27416098ec8cb8 Author: dschuyler <dschuyler@chromium.org> Date: Mon Aug 22 17:56:44 2016 [MD settings] list-button class shows actionable style mouse This CL changes list-button rows to be 'actionable'. That means that a hand mouse cursor is shown rather than an I-beam mouse cursor when the mouse is over the button. BUG= 635819 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2261803002 Cr-Commit-Position: refs/heads/master@{#413472} [modify] https://crrev.com/51071f55a9c5b147ec42660c1e27416098ec8cb8/chrome/browser/resources/settings/settings_shared_css.html
,
Aug 22 2016
,
Aug 23 2016
Verified the fix on Windows 7, MAC (10.11.6) & Ubuntu Trusty (14.04) for Google Chrome Dev Version - 54.0.2837.0 Screen-recording is attached. TE-Verified labels are attached. |
|||
►
Sign in to add a comment |
|||
Comment 1 by dschuyler@chromium.org
, Aug 19 2016