Skip to content

Add support for the relative color syntax from CSS Color 5#3676

Merged
nex3 merged 1 commit intosass:mainfrom
stof:css_relative_colors
Sep 29, 2023
Merged

Add support for the relative color syntax from CSS Color 5#3676
nex3 merged 1 commit intosass:mainfrom
stof:css_relative_colors

Conversation

@stof
Copy link
Contributor

@stof stof commented Sep 2, 2023

When using the relative color syntax, the color function is emitted as a CSS function (similar to cases using a CSS variable in arguments).

This fast track proposal updates the definition of the rgb and hsl global functions. The proposal for color spaces is also updated to reflect the new parsing.

#3673

@nex3 nex3 self-requested a review September 5, 2023 20:18
@nex3
Copy link
Contributor

nex3 commented Sep 6, 2023

Spec changes look good, but per the fast track process this should be landed along with patches to Dart Sass and the tests.

@stof
Copy link
Contributor Author

stof commented Sep 6, 2023

@nex3 I have the dart-sass change in progress locally. I will open the PR soon.

@stof
Copy link
Contributor Author

stof commented Sep 27, 2023

@nex3 PRs for sass-spec and dart-sass are now ready

When using the relative color syntax, the color function is emitted as a
CSS function (similar to cases using a CSS variable in arguments).
@stof stof force-pushed the css_relative_colors branch from 49407f2 to 1709d86 Compare September 27, 2023 22:29
@stof
Copy link
Contributor Author

stof commented Sep 29, 2023

@nex3 as you merged the implementation in dart-sass, I think this should be merged as well.

@nex3 nex3 merged commit 83a9b01 into sass:main Sep 29, 2023
@stof stof deleted the css_relative_colors branch September 29, 2023 22:13
jgerigmeyer added a commit to oddbird/sass that referenced this pull request Oct 12, 2023
* main:
  Remove `isAlphaMissing` and add "alpha" to channel name types.
  Rename types using title-case for acronyms longer than two letters in camel-case identifiers.
  Clarifying performance expectations about sass and sass-embedded (sass#3716)
  Tweak language around getters and arrays.
  Code review
  Remove unused ValueObject type
  [First-Class Mixins] Flush to spec
  Move and restructure how we specify scopes
  Fix typos in the deprecation APi proposal (sass#3708)
  Return immutable types
  Remove generic change overload
  Add first class mixins to embedded sass protobuf and JS API (sass#3674)
  Draft 1.1 changes for Color spaces JS API  (sass#3704)
  Add support for the relative color syntax from CSS Color 5 (sass#3676)
  Bump tj-actions/changed-files from 39.0.3 to 39.2.0 (sass#3697)
  Package Importer updates (sass#3699)
  Fix link in Contributing docs (sass#3705)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants