You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Creating an explainer for the work we're doing for CSS-Mixins-1. The current explainer is a publicly hosted document that contains outdated information and includes zincing (@functions). We should have a new one limited down to Mixins.
The reason will be displayed to describe this comment to others. Learn more.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Adds a new in-repo explainer for the CSS Mixins work and links it from the main README, replacing reliance on an older externally hosted explainer that includes outdated functions-focused content.
Changes:
Add a “CSS Mixins” entry to the README explainer index
Introduce a new CSSMixins/explainer.md documenting the current mixins authoring model and status
File
Description
README.md
Adds the CSS Mixins explainer to the repository’s explainer index
CSSMixins/explainer.md
New explainer document describing the current CSSWG direction for CSS mixins
Outdated claim that the draft does not define mixins
CSSMixins/explainer.md:23
This status is already outdated: the current Editor’s Draft contains normative @mixin, @apply, and @private sections. Saying that it does not define mixins misleads readers about the proposal’s current maturity; retain only the experimental-status caveat.
Incorrect ChromeStatus feature ID in link
CSSMixins/explainer.md:49
This uses the Chromium tracking issue ID as a ChromeStatus feature ID. The CSS Mixins ChromeStatus entry linked from the TAG review is 5108022310469632; point this feature-entry link there instead.
This issue also appears on line 215 of the same file.
* [Remaining questions and status](#remaining-questions-and-status)
* [References](#references)
This branch has not been deployed
No deployments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Creating an explainer for the work we're doing for CSS-Mixins-1. The current explainer is a publicly hosted document that contains outdated information and includes zincing (@functions). We should have a new one limited down to Mixins.
Old Explainer: https://css.oddbird.net/sasslike/mixins-functions/