mirror/piskel
1
0
Fork 0
mirror of https://github.com/piskelapp/piskel.git synced 2023-08-10 21:12:52 +03:00
Code Issues Projects Releases Wiki Activity

Merge pull request #396 from GMartigny/issue_#392

Browse Source 
Issue #392 : Allow to close the options drawer with the escape key.
This commit is contained in:
Julian Descottes 2016-02-03 15:57:43 +01:00
commit 365503d9b5
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/js/controller/settings/SettingsController.js
View File

@ -33,6 +33,7 @@
ns.SettingsController = function (piskelController) {
this.piskelController = piskelController;
this.closeDrawerShortcut = pskl.service.keyboard.Shortcuts.MISC.CLOSE_POPUP;
this.settingsContainer = document.querySelector('[data-pskl-controller=settings]');
this.drawerContainer = document.getElementById('drawer-container');
this.isExpanded = false;
@ -87,6 +88,8 @@
this.currentController = new settings[setting].controller(this.piskelController);
this.currentController.init();
pskl.app.shortcutService.registerShortcut(this.closeDrawerShortcut, this.closeDrawer_.bind(this));
pskl.utils.Dom.removeClass(SEL_SETTING_CLS);
var selectedSettingButton = document.querySelector('[data-setting=' + setting + ']');
if (selectedSettingButton) {
@ -109,9 +112,12 @@
};
ns.SettingsController.prototype.destroyCurrentController_ = function () {
if (this.currentController && this.currentController.destroy) {
this.currentController.destroy();
this.currentController = null;
if (this.currentController) {
pskl.app.shortcutService.unregisterShortcut(this.closeDrawerShortcut);
if (this.currentController.destroy) {
this.currentController.destroy();
this.currentController = null;
}
}
};
})();