Skip to content
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

Sync <hue> and <hue-interpolation-method> with spec #28943

Merged
merged 7 commits into from
Sep 25, 2023

Conversation

yarusome
Copy link
Contributor

@yarusome yarusome commented Sep 5, 2023

Description

This PR:

  1. updates the descriptions for increasing hue and decreasing hue for <hue-interpolation-method>;
  2. adds fallback styles for <hue-interpolation-method>;
  3. modifies the range of <hue>.

Motivation

Per w3c/csswg-drafts@2641c47 and w3c/csswg-drafts@f46b5d4.

Additional details

Related issues and pull requests

Fixes #28986.

@yarusome yarusome requested a review from a team as a code owner September 5, 2023 07:59
@yarusome yarusome requested review from chrisdavidmills and removed request for a team September 5, 2023 07:59
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Sep 5, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Sep 5, 2023

Preview URLs

Flaws (1)

Note! 1 document with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/CSS/hue
Title: <hue>
Flaw count: 1

  • bad_bcd_queries:
    • No BCD data for query: css.types.hue

(comment last updated: 2023-09-20 08:55:09)

@yarusome yarusome marked this pull request as draft September 7, 2023 08:47
@yarusome yarusome marked this pull request as ready for review September 7, 2023 08:57
@yarusome yarusome changed the title Sync <hue-interpolation-method> with spec Sync <hue> and <hue-interpolation-method> with spec Sep 7, 2023
Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

Hi @yarusome! I've provided a review and made some suggestions. I'm not sure how helpful I've been really, as this is pretty complex maths stuff and I don't really understand it. I wonder if there is someone in our peers who would do a better reviewing job than me for this one.

@yarusome yarusome marked this pull request as ready for review September 20, 2023 08:57
Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

LGTM; thanks @yarusome !

@chrisdavidmills chrisdavidmills merged commit aa86e51 into mdn:main Sep 25, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Firefox: <hue-interpolation-method> page doesn't render example
3 participants