From 43cb787b444e62bae0f60284ce01b7afce50ba54 Mon Sep 17 00:00:00 2001 From: Damien Churchill Date: Mon, 3 Oct 2011 21:11:13 +0100 Subject: webui: fix closing the connection manager Don't override the onHide method in the ConnectionManager, this was stopping the window from being able to close. --- deluge/ui/web/js/deluge-all/ConnectionManager.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/deluge/ui/web/js/deluge-all/ConnectionManager.js b/deluge/ui/web/js/deluge-all/ConnectionManager.js index 999041e91..4fda8e815 100644 --- a/deluge/ui/web/js/deluge-all/ConnectionManager.js +++ b/deluge/ui/web/js/deluge-all/ConnectionManager.js @@ -46,8 +46,8 @@ Ext.define('Deluge.ConnectionManager', { initComponent: function() { this.callParent(arguments); - this.on('hide', this.onHide, this); - this.on('show', this.onShow, this); + this.afterMethod('onHide', this.afterHide, this); + this.afterMethod('onShow', this.afterShow, this); deluge.events.on('login', this.onLogin, this); deluge.events.on('logout', this.onLogout, this); @@ -261,7 +261,7 @@ Ext.define('Deluge.ConnectionManager', { }, // private - onHide: function() { + afterHide: function() { if (this.running) window.clearInterval(this.running); }, @@ -333,7 +333,7 @@ Ext.define('Deluge.ConnectionManager', { // FIXME: Find out why this is being fired twice // private - onShow: function() { + afterShow: function() { if (!this.addHostButton) { var buttons = this.grid.query('button'); this.addHostButton = buttons[0]; -- cgit