Addy Osmani comments

Results 392 comments of


                                            Addy Osmani

Update requirejs shim text regarding underscore/backbone

Confirmed that we should rewrite that section to cover the changes introduced in November. Thank you for flagging this!

First example in chapter 2 is blank

Could you share me a zip or repo with the current code you're trying to run using the example?

Grunt-BBB examples no longer work because repo has been emptied

I think now would be the right time to move over to documenting how to use the Yeoman generator for the project instead. @SBoudrias, would you be interested in helping...

Grunt-BBB examples no longer work because repo has been emptied

@SBoudrias I know that you've got your head down working on Yeoman core these days. Would it make sense to just update the tutorial on using `grunt-bbb` over to https://github.com/backbone-boilerplate/generator-bbb...

Code examples are not readable on a kindle

Agreed. Will keep this open and try to see if we can get a better CSS override for the Kindle build in place.

backbone.js-jquerymobile-boilerplate-template: templatePartialPageID undefined

Confirmed. Thanks for looking into this. I'll work on getting this fixed.

Request : Add backbone-associations to plugin-list

Thanks. I'll check it out!. I generally make sure that I've spent time using a plugin before including but can certainly take a pass at it this weekend.

Page 37 Collections unclear use of method create

Thanks for the feedback. I'll come up with a more readable example. var TodoList = Backbone.Collection.extend({ model: Todo }); var todos = new TodoList; var item = todos.create({ title: "Buy...

Bring back in post-merge fix changes

Could you confirm if this is an issue in just the PDF/ePub generated here or in the O'Reilly version (purchased)?

Bring back in post-merge fix changes

@eugeneglova I've pushed fixes for this over into https://github.com/addyosmani/backbone-fundamentals/tree/merge-fix. Could you help double check that this update is okay? It's mostly the compiled ePub and index file we need to...