1592164 - text system color conflicts with background-clip

Status: RESOLVED WONTFIX

(Core :: CSS Parsing and Computation, defect, P3)

Description

[Emilio Cobos Álvarez (:emilio)]

Bug 570896 implemented parsing for the background-clip property in the background shorthand.

Seems like that change to the spec was reverted looking at the current draft.

This is causing issues when working on bug 1590894 because adding the text color keyword suddenly changes the meaning of the background (it starts determining background-color, not background-clip).

The css3-backgrounds draft from 13 September 2007 (8cafb8725ac5ec89deb76476b7f7d80f0c2dce1a) does contain background-clip in the background shorthand, haven't found yet when it went away.

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Looks like it was removed in https://github.com/w3c/csswg-drafts/commit/3c8802f0a0eb0526b933c91888b3a2a49b71b593.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

I can take this to unblock bug 1590894.

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1592164 - Don't parse background-clip in background-shorthand.

Bug 570896 implemented parsing for the background-clip property in the
background shorthand, but that change was later reverted in:

https://github.com/w3c/csswg-drafts/commit/3c8802f0a0eb0526b933c91888b3a2a49b71b593

This is causing issues when working on bug 1590894 because adding the text color
keyword suddenly changes the meaning of the background (it starts determining
background-color, not background-clip).

Align with other browsers and with the spec by not parsing background-clip in
the shorthand, always expanding to the initial value.

Comment 4

[Emilio Cobos Álvarez (:emilio)]

So, actually, it should parse it (and Mats pointed out in my patch the right section of the spec):

If one <box> value is present then it sets both background-origin and background-clip to that value. If two values are present, then the first sets background-origin and the second background-clip.

So now the issue is: backgrounds-3 doesn't define background-clip: text, but presumably we do want to be able to specify background-clip: text on the background shorthand, so the text color is not backwards compatible...

I'll raise this with the CSSWG, as we could go in a number of ways about this.

Comment 5

[Bill Goldstein [:aja] (UTC-6)]

Note renaming in css-color-4 of new system color Text to CanvasText
https://github.com/w3c/csswg-drafts/issues/4465#issuecomment-547994485

Comment 6

[Emilio Cobos Álvarez (:emilio)]

Yeah, this is wontfix per that resolution.