Skip to content
New issue

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

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

Already on GitHub? # to your account

Add gradients feature #1832

Merged
merged 6 commits into from
Oct 9, 2024
Merged

Conversation

jamesnw
Copy link
Collaborator

@jamesnw jamesnw commented Sep 24, 2024

I was on the fence about combining linear and radial gradients, but this mirrors Can I Use and MDN, and there isn't a separate support story.

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label Sep 24, 2024
@vwallen
Copy link
Contributor

vwallen commented Sep 25, 2024

Splitting linear/repeating-linear and radial/repeating-radial is more consistent with the conic-gradients feature, and makes sense within the gradients group

Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good, but I have a couple extremely minor textual issues to fix.

@@ -0,0 +1,31 @@
name: Gradients
description: The `linear-gradient()`, `radial-gradient()` CSS functions and their repeating counterparts create background that progress smoothly between multiple colors.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nits

Suggested change
description: The `linear-gradient()`, `radial-gradient()` CSS functions and their repeating counterparts create background that progress smoothly between multiple colors.
description: The `linear-gradient()` and `radial-gradient()` CSS functions and their repeating counterparts create backgrounds that progress smoothly between multiple colors.

name: Gradients
description: The `linear-gradient()`, `radial-gradient()` CSS functions and their repeating counterparts create background that progress smoothly between multiple colors.
spec: https://drafts.csswg.org/css-images-3/#gradients
# Todo: caniuse support needs to be reconciled for Safari and Firefox.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Todo: caniuse support needs to be reconciled for Safari and Firefox.
# TODO: caniuse support needs to be reconciled for Safari and Firefox.

@ddbeck
Copy link
Collaborator

ddbeck commented Oct 9, 2024

Oh, on the question of splitting: I'm agnostic. If you want to split it, I'll re-review. But I suspect we'll want a way to address both and a TODO saying something like we should have separate linear and radial grandients when https://github.com/web-platform-dx/web-features/issues/971 is resolved would also be fine.

@jamesnw jamesnw enabled auto-merge (squash) October 9, 2024 20:12
@jamesnw jamesnw merged commit feb5c55 into web-platform-dx:main Oct 9, 2024
3 checks passed
@jamesnw jamesnw deleted the gradient-feature branch October 10, 2024 14:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature definition Creating or defining new features or groups of features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants