Docker/documize
Docker/documize
1
0
Fork 0
mirror of https://github.com/documize/community.git synced 2025-07-20 13:49:42 +02:00
Code Releases Activity

added missing event subscription teardowns

Browse source
This commit is contained in:
Harvey Kandola 2017-01-24 08:48:27 -08:00
parent 9b2f081ef2
commit 03402d9e4d
4 changed files with 15 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

4
app/app/components/document/document-history.js
View file

@ -34,6 +34,10 @@ export default Ember.Component.extend({
this.sizeSidebar();
},
willDestroyElement() {
this.eventBus.unsubscribe('resized');
},
sizeSidebar() {
let size = $(window).height() - 200;
this.$('.document-history > .sidebar').css('height', size + "px");

2
app/app/components/document/document-sidebar.js
View file

@ -42,6 +42,8 @@ export default Ember.Component.extend(TooltipMixin, NotifierMixin, {
},
willDestroyElement() {
this.eventBus.unsubscribe('resized');
this.eventBus.unsubscribe('scrolled');
this.destroyTooltips();
},

10
app/app/components/folder/folders-list.js
View file

@ -80,6 +80,12 @@ export default Ember.Component.extend(TooltipMixin, NotifierMixin, {
this.eventBus.subscribe('scrolled', this, 'positionTool');
},
willDestroyElement() {
this.eventBus.unsubscribe('resized');
this.eventBus.unsubscribe('scrolled');
this.destroyTooltips();
},
didReceiveAttrs() {
let folders = this.get('folders');
@ -111,10 +117,6 @@ export default Ember.Component.extend(TooltipMixin, NotifierMixin, {
this.set('hasProtectedFolders', this.get('protectedFolders.length') > 0);
},
willDestroyElement() {
this.destroyTooltips();
},
positionTool() {
if (this.get('isDestroyed') || this.get('isDestroying')) {
return;

3
app/app/components/layout/zone-navigation.js
View file

@ -128,6 +128,9 @@ export default Ember.Component.extend(TooltipMixin, {
sortable.destroy();
}
this.eventBus.unsubscribe('resized');
this.eventBus.unsubscribe('pinChange');
this.destroyTooltips();
},