felixfoertsch/gnome-night-theme-switcher
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Merge branch 'fix/non-existent-keybinding-warning' into 'master'

Browse Source 
Don't remove keybinding if it wasn't added

See merge request rmnvgr/nightthemeswitcher-gnome-shell-extension!45
This commit is contained in:
Romain
2020-08-07 11:36:18 +00:00
parent be221b6c97 14a209a45a
commit bd219847f6
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/modules/TimerOndemand.js
+4
View File
@@ -41,6 +41,7 @@ var TimerOndemand = class {
constructor() { constructor() {
this._button = null; this._button = null;
this._previousKeybinding = null;
this._ondemandKeybindingConnect = null; this._ondemandKeybindingConnect = null;
this._ondemandButtonPlacementConnect = null; this._ondemandButtonPlacementConnect = null;
} }
@@ -99,6 +100,7 @@ var TimerOndemand = class {
_addKeybinding() { _addKeybinding() {
this._previousKeybinding = e.settingsManager.ondemandKeybinding;
if (!e.settingsManager.ondemandKeybinding) if (!e.settingsManager.ondemandKeybinding)
return; return;
logDebug('Adding On-demand Timer keybinding...'); logDebug('Adding On-demand Timer keybinding...');
@@ -113,10 +115,12 @@ var TimerOndemand = class {
} }
_removeKeybinding() { _removeKeybinding() {
if (this._previousKeybinding) {
logDebug('Removing On-demand Timer keybinding...'); logDebug('Removing On-demand Timer keybinding...');
main.wm.removeKeybinding('nightthemeswitcher-ondemand-keybinding'); main.wm.removeKeybinding('nightthemeswitcher-ondemand-keybinding');
logDebug('Removed On-demand Timer keybinding.'); logDebug('Removed On-demand Timer keybinding.');
} }
}
_addButton() { _addButton() {
switch (e.settingsManager.ondemandButtonPlacement) { switch (e.settingsManager.ondemandButtonPlacement) {