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

refactor: move tokens to their component directories #30765

Merged
merged 2 commits into from
Apr 4, 2025

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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