-
-
Notifications
You must be signed in to change notification settings - Fork 873
update templates section to use CLI syntax #71
Conversation
@@ -31,12 +31,12 @@ value. | |||
|
|||
### Dependencies | |||
|
|||
Imagine you want to render the full name of an `App.Person`. In this | |||
Imagine you want to render the full name of an `Person`. In this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"an Person
"
01f9c96
to
f70496f
Compare
This template will be compiled automatically and become your | ||
[application template](../../templates/the-application-template), | ||
which will be displayed on the page when your app loads. | ||
By default, adjust your [application template](/templates/the-application-template), that is created automatically for you and will be displayed on the page when your app loads. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link needs to be relative.
We'll just suffer the image for a while. Can you address the one comment and then squash this commit? |
0785d6a
to
68c28dd
Compare
@trek made link updates and squashed commit. I'm not a master of Photoshop, but I'll find a tutorial for adjusting the text within the image and try to make it look nice. When I have a chance to try, I'll submit another PR for just that change. |
@mikepmunroe that image comes from OmniGraffle, which is obviously not great for us to use. If you have the time, it'd be great to migrate that diagram to https://github.com/knsv/mermaid and create (of find) a mermaid extension for middleman. I can create a Quest Issue if you need more context than that. |
update templates section to use CLI syntax
```js | ||
App.PostController = Ember.ObjectController.extend({ | ||
```app/controllers/post.js | ||
export default Ember.ObjectController.extend({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably need to update examples that use ObjectController
.
update all docs in templates to utilize CLI syntax
there is still an image in Actions that is old syntax that need updating