New issue
Advanced search Search tips

Issue 703552 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

show color picker after "caret-color" property

Reported by l446240525@gmail.com, Mar 21 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3046.0 Safari/537.36

Steps to reproduce the problem:
.

What is the expected behavior?

What went wrong?
.

Did this work before? No 

Chrome version: 59.0.3046.0  Channel: canary
OS Version: OS X 10.10.4
Flash Version: Shockwave Flash 25.0 r0
 
33.png
16.2 KB View Download

Comment 1 Deleted

Comment 2 Deleted

Comment 3 by kdzwinel@gmail.com, Mar 21 2017

I've started it here: https://codereview.chromium.org/2764113003

Comment 4 by ajha@chromium.org, Mar 22 2017

Labels: Needs-Triage-M59
Cc: jmukthavaram@chromium.org
Labels: Needs_Feedback
Thanks for filing the issue.
l446240525@,Could you please provide us a sample html file/test case, steps  expected behaviour to triage the issue further.

Project Member

Comment 6 by bugdroid1@chromium.org, Mar 22 2017

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

commit f9f76a2c8a9757bfada489528fe3d0a8eb9ff374
Author: kdzwinel <kdzwinel@gmail.com>
Date: Wed Mar 22 09:18:14 2017

DevTools: Add 'caret-color' CSS property to _colorAwareProperties

http://i.imgur.com/i1C0xby.png

BUG= 703552 

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

[modify] https://crrev.com/f9f76a2c8a9757bfada489528fe3d0a8eb9ff374/third_party/WebKit/Source/devtools/front_end/sdk/CSSMetadata.js

Status: Fixed (was: Unconfirmed)

Comment 8 by r...@igalia.com, Mar 31 2017

Thanks for fixing this!

Would it make sense to merge this into M58?

"caret-color" is avilable since M57 and some users are missing this color picker.
Anyway I don't know if this is something that should be accepted as merge for M58 or not, just wondering.

Sign in to add a comment