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

Issue 592908 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug



Sign in to add a comment

The inherit CSS-value doesn't use the computed value in the background-color CSS property

Reported by zalawa.x...@gmail.com, Mar 8 2016

Issue description

Chrome Version: 49.0.2623.75 (Build officiel) m (64 bits)
OS: Windows
Blink: 537.36 (@76745cbfbdac04e2fe3204f8b4f96ae7dc6bfcfa)

Other browsers tested:
  Safari 9.0.3 (11601.4.4): OK
  Firefox 44.0.2: OK
  Edge 25: OK
  IE 11.0.28: OK
  Opera 12.17: OK

What steps will reproduce the problem?
1. Nest two depths of `div` HTML element.
2. Apply `background-color: currentColor` on the first depth and `background-color: inherit` on the second.
3. Apply the `color` CSS property on the two depths with a different value.

What is the expected result?
With the use of the `inherit` CSS-value in the CSS `background-color` CSS property, the background should be filled with the computed value of the `background-color` CSS property of its parent as it is defined in the CSS specification of W3C.
https://www.w3.org/TR/css3-cascade/#inherit

What happens instead?
The `inherit` CSS-value in the `background-color` CSS property uses the specified value of its parent.

 
background-color-inherit.html
402 bytes View Download
background-color-inherit.jpg
206 KB View Download
Cc: rnimmagadda@chromium.org
Components: Blink>Forms>Color
Labels: M-50 OS-Linux OS-Mac OS-Windows
Owner: fmalita@chromium.org
Status: Assigned (was: Unconfirmed)
====================================

Good Build:

44.0.2366.0    Base Position: 324746


Bad Build:

44.0.2381.0    Base Position: 326628

=====================================

Able to repro this issue on Windows 7, MAC (10.11.3) & Ubuntu Trusty (14.04) for the Google Chrome Stable Version - 49.0.2623.87

This is a regression issue broken in M44, below mentioned is the bisect info:

CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/091a9e14ae9716f30ebb17cee80db13a5dd0261d..b3ee51643703386144a3e6eca2558e72077d5fae

https://chromium.googlesource.com/chromium/blink/+log/21fff74..bb7e4b1

Suspecting Commit: a9ce0ce28c5d2f9b029fcb209358260141e2f41a

Review URL: https://codereview.chromium.org/1088613002

@fmalita: Could you please look into the issue, and if it has nothing to do with your changes and if possible please do assign it to the concerned owner.

Thank you.

Comment 3 by tkent@chromium.org, Mar 9 2016

Components: -Blink>Forms>Color Blink>CSS
This is actually correct as per the current specs. I'm not sure why other browsers haven't implemented this yet :/

This was agreed upon in 2012. https://lists.w3.org/Archives/Public/www-style/2012May/0541.html

https://drafts.csswg.org/css-color/#currentcolor-color
"The keyword currentcolor takes its value from the value of the color property on the same element. This happens at used-value time, which means that if the value is inherited, it’s inherited as currentcolor, not as the value of the color property, so descendants will use their own color property to resolve it."
Cc: fmalita@chromium.org
Owner: timloh@chromium.org
Not related to a9ce0ce28c5d2f9b029fcb209358260141e2f41a.

The change in behavior was most likely introduced in https://chromium.googlesource.com/chromium/blink/+/78db1a65a9b86eeb89885cda984a1684d50ed450.

Punting to timloh@, but it does sound like this is intended/correct behavior.
@timloh: Your comment is really interesting, I didn't think about searching into CSS4 yet :)

I understand that is the way described in the CSS Color Module Level 4, but it's still an editor's draft version and Blink is the only one which has chosen to take this way :(

We don't know if the W3C will choose an other way soon. Meanwhile, it would be good to follow the behaviour of other browsers while the CSS Color Module Level 4 is not a W3C candidate recommendation.

What do you think about that ?

Thank you for your attention on this issue.

Comment 7 by timloh@chromium.org, Mar 10 2016

Cc: tabatkins@chromium.org
This is also in css-color-3 as an errata. I think you should be filing bugs on other browsers :)

+tabatkins: Other browser vendors agreed on this and it's going to stay in the specs, right?
Yes.

Comment 9 by timloh@chromium.org, Mar 10 2016

Status: WontFix (was: Assigned)

Sign in to add a comment