Skip to content

feat(ng-add): add preview URLs for theme choices - #13723

Merged
mmalerba merged 1 commit into
angular:masterfrom
crisbeto:13708/ng-add-preview
Oct 25, 2018
Merged

feat(ng-add): add preview URLs for theme choices#13723
mmalerba merged 1 commit into
angular:masterfrom
crisbeto:13708/ng-add-preview

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Adds URLs that people can use to preview a theme, before selecting it, when going through the ng-add prompts.
Formatting is aligned with https://github.com/angular/angular-cli/blob/907ce5c94c774147c4b52870b3522aed4e087c76/packages/schematics/angular/ng-new/schema.json#L121

Fixes#13708.

A couple of notes:

@crisbetocrisbeto added the target: patch This PR is targeted for the next patch release label Oct 21, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 21, 2018

@devversiondevversion left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

"type": "boolean",
"default": true,
"description": "Whether Angular browser animations should be set up or not.",
"description": "Whether Angular browser animations should be set up.",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you update the according messages in schema.ts?

Adds URLs that people can use to preview a theme, before selecting it, when going through the ng-add prompts.
Formatting is aligned with https://github.com/angular/angular-cli/blob/907ce5c94c774147c4b52870b3522aed4e087c76/packages/schematics/angular/ng-new/schema.json#L121Fixesangular#13708.
@crisbetocrisbeto added the action: merge The PR is ready for merge by the caretaker label Oct 21, 2018
@crisbeto
crisbetoforce-pushed the 13708/ng-add-preview branch from 071aea2 to 81c0de0CompareOctober 21, 2018 14:14
@crisbetocrisbeto added the blocked This issue is blocked by some external factor, such as a prerequisite PR label Oct 21, 2018

@jelbournjelbourn left a comment

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.

LGTM

@crisbetocrisbeto removed the blocked This issue is blocked by some external factor, such as a prerequisite PR label Oct 25, 2018
@crisbeto

Copy link
Copy Markdown
MemberAuthor

Not blocked anymore.

@mmalerba
mmalerba merged commit 4fbbb9d into angular:masterOct 25, 2018
mmalerba pushed a commit that referenced this pull request Oct 26, 2018
Adds URLs that people can use to preview a theme, before selecting it, when going through the ng-add prompts.
Formatting is aligned with https://github.com/angular/angular-cli/blob/907ce5c94c774147c4b52870b3522aed4e087c76/packages/schematics/angular/ng-new/schema.json#L121Fixes#13708.
**A couple of notes:**
* Blocked until angular/material.angular.io#549 is merged and released.
* Marking this as `patch`, even though it's a `feat`, since it's basically just message changes.
atscott pushed a commit to atscott/components that referenced this pull request Nov 5, 2018
Adds URLs that people can use to preview a theme, before selecting it, when going through the ng-add prompts.
Formatting is aligned with https://github.com/angular/angular-cli/blob/907ce5c94c774147c4b52870b3522aed4e087c76/packages/schematics/angular/ng-new/schema.json#L121Fixesangular#13708.
**A couple of notes:**
* Blocked until angular/material.angular.io#549 is merged and released.
* Marking this as `patch`, even though it's a `feat`, since it's basically just message changes.
@angular-automatic-lock-bot

Copy link
Copy Markdown

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-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 10, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementtarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ng add @angular/material asks to choose theme, but there is no way to preview it

5 participants

@crisbeto@jelbourn@devversion@googlebot@mmalerba