backbone-fundamentals icon indicating copy to clipboard operation
backbone-fundamentals copied to clipboard

Problems in chapter "MarionetteJS (Backbone.Marionette)"

Open lachezar opened this issue 11 years ago • 3 comments

I am new to Backbone and I was hoping to read on Marionette, but I noticed few problematic things:

The part explaining about Marionette's Layout is very unclear to me in general. Looking in the documentation of Marionette I see different usage than the one described in this book (e.g. there is an attribute regions defined within the Layout and in the book it is missing). Also this chapter states that the Header is a Layout, but the source code afterwards extends from Marionette.ItemView.

// Layout Header View
// ------------------

Layout.Header = Marionette.ItemView.extend({

Otherwise, thanks for the efforts and the book looks great so far!

lachezar avatar Jul 06 '13 19:07 lachezar

Thank you for reporting this. I think we need to spend time reviewing the changes to the more recent versions of Marionette and adjust the chapter accordingly.

addyosmani avatar Jul 07 '13 00:07 addyosmani

I was going to report the same exact thing. Most of the chapter is easy to follow, but this part had me re-reading a couple times. Okay, I was really tired the first time I read, but still. :)

Also, maybe a general suggestion, it would help to show a diagram of the project structure up front, as well as a diagram showing call dependencies between the files/modules. And finally, simple flow diagrams showing from app start to loading up the list, adding an item, and checking an item. This sort of architectural overview would help a lot before diving into the particular pieces of it.

ambroselittle avatar Jul 27 '13 20:07 ambroselittle

This chapter requires a rewrite to bring it in line with the latest versions of Marionette. I haven't had a chance to do this yet, but will try to make out time to. Any assistance is appreciated of course :)

addyosmani avatar Oct 01 '13 08:10 addyosmani