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

docs(router): fix markdown, and example mixin cleanup #2101

Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Aug 12, 2016

  • Fixed two instances of markdown text not under :marked region (and so that text was not showing up in the generated html including the heading "Router Lifecycle Hooks").
  • Fixed: code excerpt title constructor -> isSelected.
  • Cleanup of all makeExample mixin uses.

Preliminary step to #2035: this cleanup will help make it easier to record differences with the deprecated router chapter (which must have app-relative makeExample paths).

- Fixed: markdown text not under `:marked` region (and so that text was
not showing up in the generated html).
- Fixed: code excerpt title `constructor` -> `isSelected`.
- Cleanup of all makeExample mixin uses.

This cleanup will help make it easier to record differences with the
deprecated router chapter (which must have app-relative makeExample
paths).
@chalin
Copy link
Contributor Author

chalin commented Aug 12, 2016

@brandonroberts : ready for review.

cc @wardbell @Foxandxss @kwalrath

@chalin chalin changed the title docs(router): fix markdown and example mixin cleanup docs(router): fix markdown, and example mixin cleanup Aug 12, 2016
@brandonroberts
Copy link
Contributor

brandonroberts commented Aug 12, 2016

The first code example under Query Params and Fragments is missing a header. It used to say auth-guard.service.ts (v.3) Everything else LGTM

@chalin
Copy link
Contributor Author

chalin commented Aug 12, 2016

Good catch. Fixed.

Found another instance of markdown (a heading) outside of a `:marked` region.
@chalin chalin force-pushed the chalin-ts-router-ex-mixin-cleanup-0812 branch from 060bab8 to 25d277a Compare August 12, 2016 18:01
@kwalrath kwalrath merged commit ea45782 into angular:master Aug 12, 2016
@kwalrath kwalrath deleted the chalin-ts-router-ex-mixin-cleanup-0812 branch August 12, 2016 18:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants