Skip to content
This repository was archived by the owner on Feb 22, 2018. It is now read-only.

[chap2] tweaks to match updates to tutorial prose #70

Merged
merged 3 commits into from
Feb 11, 2014
Merged

[chap2] tweaks to match updates to tutorial prose #70

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6164af6
[chap2] tweaks to match updates to tutorial prose
chalin Feb 8, 2014
f1c5aec
Merge branch 'master' into patch-chap2-comp-class-updates
chalin Feb 10, 2014
bdf753f
Copyedits: it's -> its; Controller -> controller
chalin Feb 10, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions Chapter_02/web/main.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,9 @@ import 'package:perf_api/perf_api.dart';
@MirrorsUsed(targets: const['recipe_book'], override: '*')
import 'dart:mirrors';

/* Use the NgController annotation to indicate that this class is an
* Angular Controller. The compiler will instantiate the controller if
* it finds it in the DOM.
/* Use the @NgController annotation to indicate that this class is an
* Angular Controller. Angular will instantiate the controller if
Copy link
Contributor

Choose a reason for hiding this comment

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

Controller -> controller

* it finds an element matching it's selector in the DOM.
Copy link
Contributor

Choose a reason for hiding this comment

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

it's -> its

*
* The selector field defines the CSS selector that will trigger the
* controller. It can be any valid CSS selector which does not cross
Expand Down Expand Up @@ -82,6 +82,6 @@ class MyAppModule extends Module {
}
}

main() {
void main() {
Copy link
Contributor

Choose a reason for hiding this comment

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

I tend not to include the "void", but if it's in the chapter's text (and used in other chapters), I guess we should include it here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For the record (as I had mentioned in response to another comment): the Dart Editor now declares main() as void in its auto-generated projects.

I am fixing the other issues and will commit soon.

ngBootstrap(module: new MyAppModule());
}