feat(material/schematics): Add CSS output to custom theme schematic by amysorto · Pull Request #30004 · angular/components · GitHub
Skip to content

feat(material/schematics): Add CSS output to custom theme schematic#30004

Merged
amysorto merged 1 commit into
angular:mainfrom
amysorto:schematic-css-output
Nov 22, 2024
Merged

feat(material/schematics): Add CSS output to custom theme schematic#30004
amysorto merged 1 commit into
angular:mainfrom
amysorto:schematic-css-output

Conversation

@amysorto

@amysorto amysorto commented Nov 13, 2024

Copy link
Copy Markdown
Contributor

Fixes #29363

@angular-robot angular-robot Bot added the detected: feature PR contains a feature commit label Nov 13, 2024
@amysorto amysorto force-pushed the schematic-css-output branch 2 times, most recently from 06fd9dd to c844199 Compare November 15, 2024 18:32
@amysorto amysorto added the target: patch This PR is targeted for the next patch release label Nov 15, 2024
@amysorto amysorto marked this pull request as ready for review November 15, 2024 18:40
@amysorto amysorto requested a review from a team as a code owner November 15, 2024 18:40
@amysorto amysorto requested review from wagnermaciel and removed request for a team November 15, 2024 18:40
@amysorto amysorto added dev-app preview When applied, previews of the dev-app are deployed to Firebase and removed dev-app preview When applied, previews of the dev-app are deployed to Firebase labels Nov 15, 2024
@github-actions

Copy link
Copy Markdown

## Background
This schematic allows users to create new Material 3 theme palettes based
on custom colors by using [Material Color Utilities](https://github.com/material-foundation/material-color-utilities).
This is an alternative to using the available [predefined theme palettes](https://material.angular.io/guide/theming#prebuilt-color-palettes).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This theming guide link no longer points to a specific section (in next.material.angular.io) - this and other spots in the README that need to be adjusted

The new theming guide doesnt really list out the prebuilts anymore. I feel like they might be outdated now that its so easy to add your own theme with mat.theme

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was looking at the rc guide but just removed the rc in the url since the target for this PR is patch.

So this link for that is https://rc.material.angular.io/guide/theming#prebuilt-color-palettes, which does have the list of the prebuilt palettes. Should this still be removed?

Comment thread src/material/schematics/ng-generate/theme-color/README.md Outdated
Comment thread src/material/schematics/ng-generate/theme-color/README.md Outdated
Comment thread src/material/schematics/ng-generate/theme-color/README.md Outdated
@amysorto amysorto force-pushed the schematic-css-output branch from 772e236 to cd40d8e Compare November 21, 2024 16:16
@amysorto amysorto removed the request for review from wagnermaciel November 22, 2024 18:53
@amysorto amysorto added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Nov 22, 2024
@amysorto amysorto merged commit f47f5f9 into angular:main Nov 22, 2024
@angular-automatic-lock-bot

Copy link
Copy Markdown

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Dec 23, 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 detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(theming): Replace SASS-Based theming with global CSS variables

2 participants