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(material/tabs): switch to tokens API #26881

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Apr 4, 2023

Switches the tabs to use the new tokens API.

@crisbeto crisbeto added P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: major This PR is targeted for the next major release labels Apr 4, 2023
@crisbeto crisbeto force-pushed the tabs-tokens branch 3 times, most recently from 2ed92f4 to 52b2cc6 Compare April 4, 2023 13:03
Switches the tabs to use the new tokens API.
@crisbeto crisbeto requested a review from andrewseguin April 5, 2023 11:39
@crisbeto crisbeto marked this pull request as ready for review April 5, 2023 11:39
@crisbeto crisbeto requested a review from mmalerba as a code owner April 5, 2023 11:39
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Apr 5, 2023
@crisbeto crisbeto added target: minor This PR is targeted for the next minor release and removed target: major This PR is targeted for the next major release labels Apr 6, 2023
@angular-robot angular-robot bot merged commit cb29697 into angular:main Apr 6, 2023
crisbeto added a commit to crisbeto/material2 that referenced this pull request Apr 14, 2023
Switches the tabs to use the new tokens API.
angular-robot bot pushed a commit that referenced this pull request Apr 14, 2023
* refactor(material/tabs): switch to tokens API (#26881)

Switches the tabs to use the new tokens API.

* refactor(material/tabs): introduce new tokens (#26901)

Adds new tokens to the tabs to make it easier to implement the gmat variant.

* build: update ng add standalone integration test (#26894)

The layout for a standalone project has changed since we added our integration test.

* fix(material/tabs): remove background color from tabs (#26905)

Resolves a recent regression where tabs no longer have a `background: none` which can cause them to be grey when used on a `<button>` element.

* fix(cdk/layout): resolve CSP errors

The `MediaMatcher` needs to insert a `style` tag in some cases. These changes use the new `CSP_NONCE` token to avoid these issues.

* fix(cdk-experimental/table-scroll-container): avoid CSP errors

Adds a nonce to the `style` tag generated by the table scroll container to avoid CSP errors.

* fix(cdk-experimental/column-resize): avoid CSP errors

Uses the new `CSP_NONCE` token to avoid errors when inserting `style` tags dynamically in the column resize package.

* fix(material/legacy-progress-spinner): avoid CSP errors

Uses the new `CSP_NONCE` injection token to avoid errors when inserting a `style` tag dynamically for the spinner animation.
crisbeto added a commit to crisbeto/material2 that referenced this pull request Apr 14, 2023
Switches the tabs to use the new tokens API.
angular-robot bot pushed a commit that referenced this pull request Apr 14, 2023
Switches the tabs to use the new tokens API.
@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 Jun 6, 2023
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 P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants