fix window decorations activate/destroy on setting change

hardpixel · Jan 15, 2018 · 4bc88ec · 4bc88ec
1 parent a3af262
commit 4bc88ec
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/[email protected]/modules/windowDecoration.js b/[email protected]/modules/windowDecoration.js
@@ -169,20 +169,25 @@ var WindowDecoration = new Lang.Class({
 
   _toggle: function() {
     this._enabled = this._settings.get_enum('hide-window-titlebars');
-    this._enabled != 0 ? this._create() : this.destroy();
+    this._enabled != 0 ? this._activate() : this.destroy();
   },
 
-  _create: function() {
+  _activate: function() {
     Mainloop.idle_add(Lang.bind(this, this._addUserStyles));
     Mainloop.idle_add(Lang.bind(this, this._undecorateWindows));
 
-    this._connectSignals();
+    if (!this._activated) {
+      this._activated = true;
+      this._connectSignals();
+    }
   },
 
   destroy: function() {
     Mainloop.idle_add(Lang.bind(this, this._removeUserStyles));
     Mainloop.idle_add(Lang.bind(this, this._decorateWindows));
 
     this._disconnectSignals();
+
+    this._activated = false;
   }
 });