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

docs(dart): cleanup 'live example' links#1449

Merged
kwalrath merged 2 commits intoangular:masterfrom
chalin:chalin-doc-cleanup-eg-links-0520
May 20, 2016
Merged

docs(dart): cleanup 'live example' links#1449
kwalrath merged 2 commits intoangular:masterfrom
chalin:chalin-doc-cleanup-eg-links-0520

Conversation

@chalin
Copy link
Copy Markdown
Contributor

@chalin chalin commented May 20, 2016

This fixes various issues such as:

  • Missing space in Dart version between the live demo and demo source links.
  • Live example link directing to Plunker for Dart.
  • Copyedits.

The main mixins (liveExampleLink and liveExampleLink2) used to support the generation of links now assume reasonable default values for their arguments, and in most cases the defaults are used.

This fixes various issues such as:
- missing space in Dart version between the live demo and demo source
links.
- Live example link directing to Plunker for Dart.

The main mixins (`liveExampleLink`*) used to support the generation of
links now assume reasonable default values for their arguments, and in
most cases the defaults are used.
@chalin
Copy link
Copy Markdown
Contributor Author

chalin commented May 20, 2016

@kwalrath : this impacts mainly the generated Dart docs (the TS docs will be essentially the same).

@kwalrath
Copy link
Copy Markdown
Contributor

I'll take a look. @wardbell, do you also want to review this?

@wardbell
Copy link
Copy Markdown
Contributor

Nah. Go for it!

@kwalrath
Copy link
Copy Markdown
Contributor

LGTM! I'll squash and merge.

@kwalrath kwalrath merged commit 6d2d52c into angular:master May 20, 2016
@chalin chalin deleted the chalin-doc-cleanup-eg-links-0520 branch May 20, 2016 23:41
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