Prevent zombie view in router.js about backboneboilerplate HOT 7 CLOSED

Thanks Thomas, you are exactly right. I am working on the View Manager write now that controls the life cycle of views and stops zombie views, will be pushing in the next few hours. Thanks again!

from backboneboilerplate.

Hi,

I think it would be a nice improvement when we return 'this' on the render method.

ChildPage = Backbone.View.extend {
render: function() {
// do something
return this;
}
}

Sometime, from master page we want to append a child page, we could do:
childPage = new ChildPage()
$(element).append(childPage.render().el)

from backboneboilerplate.

I've also had issues with the current View Manager, which, while great, doesn't clean up properly. I came across this after loading views where events defined in (events.js) were being triggered multiple times. To work around this, I'm using undelegateEvents() on the previous view before creating a new one.

I'm aware that the VM isn't fully implemented yet, but any plans on setting this up?

from backboneboilerplate.

on second thought undelegateEvents is probably the only default cleanup a view manager should do and also calling clean() if it has been defined. What do you think?

from backboneboilerplate.

I'm certainly no expert, but that's just what I'm doing! Sounds good :)

Also, on the subject of VM - is it supposed to re-use already created views when called again? As currently, it doesn't seem to, and just creates a new view when create is called. In fact, views[name] is never defined.

Thanks for your quick response by the way!

from backboneboilerplate.

So in the view manager the views should always be created fresh, if you want to maintain a view you can do so using your applications logic.

Some goals I had for the view manager were

• clean up !important
• managing children / nested views
• a basic signalling of view creation to the app
• eventually more advanced event delegation e.g. the ability to make all event listeners on a view dormant etc
• handle dynamic reusable views such as table rows

For now I've changed the unbind() to undelegateEvents().

from backboneboilerplate.

Ok, that clears that up - sounds good. Thanks mate!

from backboneboilerplate.