Devtools contrast ratio widget (within color picker) not available when color is CSS variable
Reported by
mike.her...@lullabot.com,
Jul 4
|
||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.99 Safari/537.36 Steps to reproduce the problem: 1. Create text using CSS Variable 2. Inspect the text. Click the color block to open up the color picker 3. Note that the Contrast Ratio widget is not present. 4. If you set the text color using a normal hex value, the contrast ratio does exist. Note that I checked both stable and canary releases. What is the expected behavior? What went wrong? I expected the contrast ratio widget to be there Did this work before? N/A Chrome version: 67.0.3396.99 Channel: canary OS Version: OS X 10.11.6 Flash Version:
,
Jul 6
reporter@ - Thanks for filing the issue...!! Could you please provide a sample test file/url to test the issue from TE-end. This will help us in triaging the issue further. Thanks...!!
,
Jul 6
Here you go. Thanks :) https://codepen.io/mherchel/pen/wXLEmO
,
Jul 6
Thank you for providing more feedback. Adding the requester to the cc list. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 9
Good bug. thanks
,
Oct 12
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/548c459fb7741b83bd517b12882f533b04a5513e commit 548c459fb7741b83bd517b12882f533b04a5513e Author: Joel Einbinder <einbinder@chromium.org> Date: Fri Oct 12 22:17:09 2018 [DevTools] fix(styles): color contrast info for css variables Bug: 860286 Change-Id: Iec7cf205d6b240cb54b113690e61414e12685df9 Reviewed-on: https://chromium-review.googlesource.com/c/1278160 Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org> Commit-Queue: Joel Einbinder <einbinder@chromium.org> Cr-Commit-Position: refs/heads/master@{#599378} [modify] https://crrev.com/548c459fb7741b83bd517b12882f533b04a5513e/third_party/blink/renderer/devtools/front_end/elements/StylePropertyTreeElement.js |
||||
►
Sign in to add a comment |
||||
Comment 1 by krajshree@chromium.org
, Jul 5