Skip to content

Conversation

andrewseguin
Copy link
Contributor

@andrewseguin andrewseguin commented Apr 1, 2025

Moves tokens out of core/tokens and into their respective component directories.

Also uses a new token-utils.slot() function that replaces create-tokens-slot:

// Before:
.mat-comp {
  @include token-utils.create-token-slot(border-radius, container-shape);
}

// After:
.mat-comp {
  border-radius: token-utils.slot(container-shape);
}

@andrewseguin andrewseguin added the target: major This PR is targeted for the next major release label Apr 1, 2025
@andrewseguin andrewseguin requested a review from a team as a code owner April 1, 2025 19:28
@andrewseguin andrewseguin requested review from crisbeto, mmalerba and wagnermaciel and removed request for a team April 1, 2025 19:28
@mmalerba mmalerba removed their request for review April 2, 2025 03:34
@andrewseguin andrewseguin requested a review from a team as a code owner April 2, 2025 11:02
@josephperrott josephperrott removed the request for review from a team April 2, 2025 15:55
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Apr 4, 2025
@andrewseguin andrewseguin merged commit 965aadf into angular:main Apr 4, 2025
20 of 22 checks passed
mistrykaran91 pushed a commit to mistrykaran91/components that referenced this pull request Apr 7, 2025
* refactor: move token files in their component directories

* refactor: correct card prefix
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators May 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants