- From: Kaelig Deloumeau-Prigent via GitHub <sysbot+gh@w3.org>
- Date: Thu, 22 Aug 2024 19:37:12 +0000
- To: public-design-tokens-log@w3.org
kaelig has just merged drwpow's pull request 246 for https://github.com/design-tokens/community-group: == Add layered shadow support, add support for inset shadows (non-breaking changes) == ## Summary This proposes the following changes to `$type: shadow` tokens: - Allows arrays for `$value` - Adds `inset` (boolean) property ## Reasoning This PR proposes that the following discussions be resolved like so: - #100: ✅ **Accepted** (all proposals that I saw) ## Pros - Adds layered shadow support ([example](https://shadows.brumm.af/)) without breaking existing shadow tokens - Adds support for inset shadows, also without breaking backwards compatibility ## Cons - Arrays vs non-arrays adds a minor bit of tooling annoyance, but nothing significant ## Alternatives - We could just enforce “always arrays,” but IMO that’s not necessary to put that burden on the people writing JSON. The spec allows for other “syntactic sugar” and this seems like a good <abbr title="Quality of Life">QoL</abbr> addition - We could make `inset` required, at the expense of breaking backward compatibility - Instead of `"inset": [boolean]`, we could have something like `"position": "inner"` (string enum), but I can’t think of any other value this would be (what is there other than “inside” and “outside”?). Seems like designing for something no one has asked for. ## Notes - Other token types such as gradients and cubic béziers already have arrays for values, so this doesn’t introduce inconsistencies in the design See https://github.com/design-tokens/community-group/pull/246 -- Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Thursday, 22 August 2024 19:37:13 UTC