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

docs(template-syntax/dart): enhancements to example code #2051

Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Aug 8, 2016

Enhancements to NgStyle section in support of its API docs.

  • Add feature supporting interactive update of a paragraph’s style.
  • Add full type declarations.
  • Replace bogus implementation of getStyles().

@chalin
Copy link
Contributor Author

chalin commented Aug 8, 2016

cc @kwalrath @thso @matanlurey : ready for review and merge.
cc @wardbell @Foxandxss : this is a Dart-only enhancement.

@chalin chalin force-pushed the chalin-dart-template-syntax-ngstyle-0807 branch from 0153bfb to c75204c Compare August 8, 2016 23:02
@kwalrath
Copy link
Contributor

kwalrath commented Aug 8, 2016

LGTM, but an engineer should review the changes, if possible.

@thso @matanlurey

@chalin chalin force-pushed the chalin-dart-template-syntax-ngstyle-0807 branch from c75204c to 080e88f Compare August 10, 2016 11:52
chalin added 2 commits August 10, 2016 15:36
Enhancements to `NgStyle` section in support of its API docs.
- Add feature supporting interactive update of a paragraph’s style.
- Add full type declarations.
- Replace bogus implementation of `getStyles()`.
@chalin chalin force-pushed the chalin-dart-template-syntax-ngstyle-0807 branch from 080e88f to 362de13 Compare August 10, 2016 22:36
@kwalrath
Copy link
Contributor

We didn't get the engineer review, but the changes work and LGTM, so I'm merging.

@kwalrath kwalrath merged commit 596b686 into angular:master Aug 11, 2016
@kwalrath kwalrath deleted the chalin-dart-template-syntax-ngstyle-0807 branch August 11, 2016 18:48
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.

3 participants