Fix clear button not removing hotkey (#550)

Previously, clicking "clear" does not change how the hotkeys are
persisted, but it does remove TriggerMode.HOTKEY from the item, which is
persisted as an empty "modes" array in the json. When autokey is
reloaded, the empty "modes" array means the hotkeys in that item are
ignored. With the aforementioned lines adding that triggermode back, the
hotkeys that are persisted are seen and taken up as hotkeys.

Priviously, saving in the gui didn't touch the hotkey if the hotkey box
wasn't enabled.
Now: saving in the gui calls item.unset_hotkey if hotkey box is
disabled.

lib/autokey/gtkui/configwindow.py

@@ -183,6 +183,8 @@ def save(self):
             self.abbrDialog.save(self.currentItem)
         if self.hotkeyEnabled:
             self.hotkeyDialog.save(self.currentItem)
+        else:
+            self.currentItem.unset_hotkey()
         if self.filterEnabled:
             self.filterDialog.save(self.currentItem)
         else:

lib/autokey/model/abstract_hotkey.py

@@ -45,9 +45,6 @@ def set_hotkey(self, modifiers, key):
         modifiers.sort()
         self.modifiers = modifiers
         self.hotKey = key
-        # TODO removing the next two lines fixes a bug where the "clear" button
-        # for hotkeys does not persist the clearance between restarts. Unsure
-        # whether these lines are needed, or why they fix the bug.
         if key is not None and TriggerMode.HOTKEY not in self.modes:
             self.modes.append(TriggerMode.HOTKEY)
 

lib/autokey/qtui/settingswidget.py

@@ -87,6 +87,8 @@ def save(self):
             self.abbr_settings_dialog.save(self.current_item)
         if self.hotkey_enabled:
             self.hotkey_settings_dialog.save(self.current_item)
+        else:
+            self.currentItem.unset_hotkey()
         if self.window_filter_enabled:
             self.window_filter_dialog.save(self.current_item)
         else: