##// END OF EJS Templates
- remove second line in create_child_view...
Jonathan Frederic -
Show More
@@ -257,9 +257,7 b' function(WidgetManager, Underscore, Backbone){'
257 257 // TODO: this is hacky, and makes the view depend on this cell attribute and widget manager behavior
258 258 // it would be great to have the widget manager add the cell metadata
259 259 // to the subview without having to add it here.
260 options = options || {};
261 options.cell = this.options.cell;
262 var child_view = this.model.widget_manager.create_view(child_model, options);
260 var child_view = this.model.widget_manager.create_view(child_model, options || {});
263 261 this.child_views[child_model.id] = child_view;
264 262 return child_view;
265 263 },
@@ -267,8 +265,10 b' function(WidgetManager, Underscore, Backbone){'
267 265 delete_child_view: function(child_model, options) {
268 266 // Delete a child view that was previously created using create_child_view.
269 267 var view = this.child_views[child_model.id];
270 delete this.child_views[child_model.id];
271 view.remove();
268 if (view !== undefined) {
269 delete this.child_views[child_model.id];
270 view.remove();
271 }
272 272 },
273 273
274 274 do_diff: function(old_list, new_list, removed_callback, added_callback) {
General Comments 0
You need to be logged in to leave comments. Login now