Skip to content

[css-backgrounds] <color> in <shadow> should default to currentcolor #2766

Closed

Description

@upsuper

Currently the spec says:

Specifies the color of the shadow. If the color is absent, the used color is taken from the color property.

It seems to behave like currentcolor nowadays, but it would have different behavior in terms of interpolating. I think we should probably make it default to currentcolor instead. I see no reason we need a separate no-color value case.

(It's not clear to me whether it should go to level 3 or 4.)

Activity

  1. svgeesus commented on Jun 14, 2018

    @svgeesus
    Collaborator

    (It's not clear to me whether it should go to level 3 or 4.)

    When we get agreement on this, it would be a change to color 4 and an erratum on color 3.

  2. upsuper commented on Jun 20, 2018

    @upsuper
    MemberAuthor

    Let's put it into agenda... but next telecon is not apac timezone one which can be hard for me to take part in so maybe sydney f2f...

  3. upsuper commented on Jun 20, 2018

    @upsuper
    MemberAuthor

    This also affects drop-shadow() in Filter Effects, which has similar statement:

    the missing used color is taken from the color property.

    It should use currentcolor as well, I suppose.

  4. svgeesus commented on Jul 2, 2018

    @svgeesus
    Collaborator

    I agree that referencing currentColor is the correct way to go here for both shadow and drop-shadow().

  5. css-meeting-bot commented on Jul 3, 2018

    @css-meeting-bot
    Member

    The Working Group just discussed <color> in <shadow> should default to currentColor, and agreed to the following:

    • RESOLVED: Accept the proposal
    • RESOLVED: drop-shadow and shadow will also default to currentColor
    The full IRC log of that discussion <myles> Topic: <color> in <shadow> should default to currentColor
    <myles> Github: https://github.com//issues/2766
    <myles> Fantasai: any objections?
    <chris> rrsagent, here
    <RRSAgent> See https://www.w3.org/2018/07/03-css-irc#T23-20-25
    <myles> Rossen: We need to make sure the people who care have a chance to look
    <myles> xidorn: the issue is that if it isn’t specify, the used color is taken from the current property
    <myles> xidorn: I think the text should just be “it defaults to currentColor”
    <myles> dbaron: the spec may have evolved from something that was more flexible
    <myles> chris: let’s say it’s user defined
    <myles> Rossen: are we okay with the proposed resolution?
    <myles> Rossen: any objections? <silence> resolved
    <chris> s/let/(sarcastically) let/
    <myles> RESOLVED: Accept the proposal
    <Rossen> s/Accept the proposal/<color> in <shadow> should default to currentcolor
    <myles> chris: There’s also the same problem with drop-shadow as well as shadow
    <myles> leaverou: it doesn’t work that way already?
    <myles> Rossen: any additional comments?
    <myles> Rossen: <silence> Any objections?
    <myles> RESOLVED: drop-shadow and shadow will also default to currentColor
  6. added
    Commenter SatisfiedCommenter has indicated satisfaction with the resolution / edits.
    on Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions