Skip to content
This repository was archived by the owner on Dec 18, 2024. It is now read-only.

feat: add the ability to link to the site with a particular theme #549

Merged
merged 1 commit into from
Oct 24, 2018

Conversation

crisbeto
Copy link
Member

* Reworks the theming setup to store only the current theme's name, rather than the entire object in `localStorage`. This is more robust for the cases where we want to change the theme object's interface.
* Adds the ability to link people to the docs site with a particular theme (e.g. https://material.angular.io?theme=purple-green). This allows us to preview the theme. For example we can use it for angular/components#13708 to show people what the theme would look like when they're prompted in a schematic.
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

LGTM. Misses some tests for the Route stuff, but probably not that necessary.

@jelbourn jelbourn merged commit 80addca into angular:master Oct 24, 2018
mmalerba pushed a commit to angular/components that referenced this pull request Oct 25, 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#L121

Fixes #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.
mmalerba pushed a commit to angular/components 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#L121

Fixes #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#L121

Fixes angular#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.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants