Skip to content

[css-color-5] Consider allowing multiple color-adjusters to be used in color-adjust() #6054

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
weinig opened this issue Feb 28, 2021 · 3 comments

Comments

@weinig
Copy link
Contributor

weinig commented Feb 28, 2021

In the current CSS Color 5 draft's color-adjust() function, https://drafts.csswg.org/css-color-5/#coloradjust, only 0 or 1 color-adjusters are allowed per the grammar. While clearly one could nest color-adjust() calls, it seems like it would be more consistent with color-mix() to allow multiple adjusters (within the picked colorspace).

@smfr smfr added the css-color-5 Color modification label Mar 1, 2021
@LeaVerou
Copy link
Member

LeaVerou commented Mar 2, 2021

Note that if we allow multiple adjusters we'd need to resolve these against color-adjust(): #6045 #6048

Though an easy way out would be to define multiple adjusters as sugar over nested color-adjust() calls.

@argyleink
Copy link
Contributor

Can the HTTParchive data help us here? see how often there's multiple adjustments?
or, I do like the idea of shipping the feature, that requires nesting for multiple adjustments, now and adding sugar later.

@svgeesus
Copy link
Contributor

Closed by WG resolution to drop color-adjust()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants