diff --git a/src/abstract/ModuleDomainViews.ts b/src/abstract/ModuleDomainViews.ts index a23cde8c2..a78b188b5 100644 --- a/src/abstract/ModuleDomainViews.ts +++ b/src/abstract/ModuleDomainViews.ts @@ -43,7 +43,7 @@ export default abstract class ModuleDomainViews< * @private * */ private add(model: ModuleModel, fragment?: DocumentFragment) { - const { reuseView, viewCollection, itemsView = {} } = this; + const { reuseView, viewCollection } = this; var frag = fragment || null; var typeField = model.get(this.itemType); let view; @@ -64,7 +64,7 @@ export default abstract class ModuleDomainViews< } render() { - var frag = document.createDocumentFragment(); + const frag = document.createDocumentFragment(); this.clearItems(); this.$el.empty(); diff --git a/src/canvas/view/FramesView.ts b/src/canvas/view/FramesView.ts index 23d5b0726..afecb73d8 100644 --- a/src/canvas/view/FramesView.ts +++ b/src/canvas/view/FramesView.ts @@ -10,7 +10,6 @@ export default class FramesView extends ModuleDomainViews constructor(opts = {}, config: any) { super(opts, true); - //console.log(this.collection) this.listenTo(this.collection, 'reset', this.render); this.canvasView = config.canvasView; this._module = config.module; @@ -24,6 +23,13 @@ export default class FramesView extends ModuleDomainViews const { $el, ppfx } = this; $el.attr({ class: `${ppfx}frames` }); } + + clearItems() { + const items = this.viewCollection || []; + items.forEach(item => item.remove()); + this.viewCollection = []; + } + protected renderView(item: any, type: string) { return new FrameWrapView(item, this.canvasView); }