getComputedStyle().cursor does not include custom cursors coordinates
Reported by
thembr...@gmail.com,
May 10 2016
|
||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.7 Safari/537.36 Example URL: http://output.jsbin.com/romugal/1 Steps to reproduce the problem: 1. Open http://output.jsbin.com/romugal/1 The example element has a css cursor attribute: cursor: url(...) 10 10, auto; What is the expected behavior? What went wrong? Open developer console and examine log output window.getComputedStyle(el).cursor is: "url(...), auto" (without the x/y coordinates) while "el.style.cursor" contains the x/y coordinates as expected: "url(...), 10 10 auto" Does it occur on multiple sites: Yes Is it a problem with a plugin? No Did this work before? N/A Does this work in other browsers? Yes Chrome version: 51.0.2704.7 Channel: n/a OS Version: Flash Version: Shockwave Flash 21.0 r0 Works as expected in Firefox.
,
May 10 2016
,
May 10 2016
,
May 11 2016
Taking a look.
,
Aug 18 2016
Could you please provide an update on this? Thank you!
,
Aug 18 2016
Happy to work on this if you didn't get a chance rob. :)
,
Aug 18 2016
Thanks, I did a first patch but of course I hit some tricky corner cases :) I can share that patch if you want, I'll make this issue available as I have higher prio work atm.
,
Aug 19 2016
Totally, please send it my way :) I'll happily take this on.
,
Sep 7 2016
Has anyone actually tested this? It works fine in Chrome stable, 53.0.2785.92 (64-bit) on Ubuntu. Adding a test for it.
,
Sep 7 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3e925c0b98486847e48dafc2aeb5878ac28f4ef1 commit 3e925c0b98486847e48dafc2aeb5878ac28f4ef1 Author: sashab <sashab@chromium.org> Date: Wed Sep 07 05:26:55 2016 Added a test for custom cursor coordinates, which already works Added a test for custom cursor coordinates, which is already implemented but has been reported in the specified bug. BUG= 610635 Review-Url: https://codereview.chromium.org/2314293003 Cr-Commit-Position: refs/heads/master@{#416850} [add] https://crrev.com/3e925c0b98486847e48dafc2aeb5878ac28f4ef1/third_party/WebKit/LayoutTests/fast/css/getComputedStyle/computed-style-cursor.html
,
Sep 7 2016
@sashab seems you fixed it earlier indeed! See https://codereview.chromium.org/2028493002 However when I started working on a patch that was not in yet I think. Anyway looks like this bug can be closed.
,
Sep 8 2016
:) Yay! |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by tasak@google.com
, May 10 2016