Skip to content

refactor: create mixins for core classes #29786

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

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

wagnermaciel
Copy link
Contributor

  • This is an intermediary step so we can roll out calls to these empty mixins progressively.
  • Once all of the instances of mat.core have been updated to also call mat.app-background and mat.elevation-classes we can move the styles over without breaking any clients

* This is an intermediary step so we can roll out calls
  to these empty mixins progressively.
* Once all of the instances of mat.core have been updated
  to also call mat.app-background and mat.elevation-classes
  we can move the styles over without breaking any clients
@wagnermaciel wagnermaciel requested a review from a team as a code owner September 24, 2024 21:42
@wagnermaciel wagnermaciel requested review from amysorto and mmalerba and removed request for a team September 24, 2024 21:42
@wagnermaciel wagnermaciel added action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release labels Sep 24, 2024
@wagnermaciel wagnermaciel removed the request for review from amysorto September 24, 2024 22:04
@wagnermaciel wagnermaciel merged commit 6315e79 into angular:main Sep 24, 2024
23 of 25 checks passed
@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 Oct 25, 2024
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