Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion for ch 7 "Sections" topic #66

Open
reybango opened this issue Feb 16, 2016 · 1 comment
Open

Suggestion for ch 7 "Sections" topic #66

reybango opened this issue Feb 16, 2016 · 1 comment

Comments

@reybango
Copy link

In ch7, there's a section that dives into the concept of "Sections" within your view. Cool way to use Handlebars helpers but I wanted to offer some feedback to make it a little clearer:

  1. You should explain what options.fn() does. I had to go to the Handlebars site to find a reference to it and even then it wasn't very clear on what it actually does.
  2. The section in the book ends without explaining where to add the _section.head & _section.jquery code. I assumed it was main.handlebars so I added it there.
  3. There were no instructions on how to reference the new 'jquery-test.handlebars' template so I just created a simple route to call it. The book should mention something about that.
@EthanRBrown
Copy link
Owner

Thanks for the feedback, Ray, those are all good points. I know you've got some additional PRs, which I will get to soon! I am going to leave this open because I'll use it as a reminder to make changes to the copy in the book.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants