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

docs(dev guide): structural-directives - ts minor edits, dart first version #1277

Merged
merged 1 commit into from
May 4, 2016

Conversation

chalin
Copy link
Contributor

@chalin chalin commented May 3, 2016

  • copyedits and other minor edits for TS
  • a first version of the chapter for Dart

@chalin chalin changed the title docs(dev guide): structural-directives ts copyedits, dart first version [review-pending] docs(dev guide): structural-directives ts copyedits, dart first version May 3, 2016
@chalin
Copy link
Contributor Author

chalin commented May 3, 2016

Ready for review and merge.
@filipesilva : can you review the TS file?
@thso : the Dart version is pretty much just the TS version but with Dart examples.

@chalin chalin changed the title [review-pending] docs(dev guide): structural-directives ts copyedits, dart first version [review-pending] docs(dev guide): structural-directives - ts minor edits, dart first version May 4, 2016
@@ -163,7 +166,7 @@ figure.image-display
We can confirm these effects by wrapping the middle "hip" of the phrase "Hip! Hip! Hooray!" within a `<template>` tag.
+makeExample('structural-directives/ts/app/structural-directives.component.html', 'template-tag')(format=".")
:marked
The display is a 'Hip!' short of perfect enthusiasm. The DOM effects are different when Angular is control.
The display is a 'Hip! Hooray!', short of perfect enthusiasm. The DOM effects are different when Angular is control.
Copy link

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

Choose a reason for hiding this comment

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

Think there is an issue with end of the sentence. Do we mean: 'when Angular IS in control'?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. Fixed.

@thso
Copy link
Contributor

thso commented May 4, 2016

Reviewing this now.

@thso
Copy link
Contributor

thso commented May 4, 2016

LGTM

…ersiondocs(dev guide): structural-directives - ts minor edits, dart first version

- copyedits and other minor edits for TS
- a first version of the chapter for Dart
@chalin chalin force-pushed the chalin-guide-struct-dir-0503 branch from f1000f3 to 1b2454a Compare May 4, 2016 11:25
@chalin chalin changed the title [review-pending] docs(dev guide): structural-directives - ts minor edits, dart first version docs(dev guide): structural-directives - ts minor edits, dart first version May 4, 2016
@thso thso merged commit ce6c645 into angular:master May 4, 2016
@chalin chalin deleted the chalin-guide-struct-dir-0503 branch May 4, 2016 12:37
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