Merge pull request #184 from unsecretised/remove-hotkey-hot-reloading

Remove hotkey reloading on config reload

Author: unsecretised

src/app/tile.rs

@@ -10,7 +10,7 @@ use crate::{app::apps::App, platform::default_app_paths};
 
 use arboard::Clipboard;
 use global_hotkey::hotkey::HotKey;
-use global_hotkey::{GlobalHotKeyEvent, GlobalHotKeyManager, HotKeyState};
+use global_hotkey::{GlobalHotKeyEvent, HotKeyState};
 
 use iced::futures::SinkExt;
 use iced::futures::channel::mpsc::{Sender, channel};
@@ -136,14 +136,6 @@ pub struct Hotkeys {
     pub clipboard_hotkey: HotKey,
 }
 
-impl Hotkeys {
-    pub fn register(&self) {
-        let manager = GlobalHotKeyManager::new().unwrap();
-        manager.register(self.toggle).ok();
-        manager.register(self.clipboard_hotkey).ok();
-    }
-}
-
 impl Tile {
     /// This returns the theme of the window
     pub fn theme(&self, _: window::Id) -> Option<Theme> {

src/app/tile/update.rs

@@ -26,7 +26,6 @@ use crate::app::{Message, Page, tile::Tile};
 use crate::calculator::Expr;
 use crate::commands::Function;
 use crate::config::Config;
-use crate::tile::Hotkeys;
 use crate::unit_conversion;
 use crate::utils::is_valid_url;
 use crate::{app::ArrowKey, platform::focus_this_app};
@@ -267,19 +266,6 @@ pub fn handle_update(tile: &mut Tile, message: Message) -> Task<Message> {
             new_options.extend(App::basic_apps());
             new_options.par_sort_by_key(|x| x.display_name.len());
 
-            tile.hotkeys = Hotkeys {
-                toggle: new_config
-                    .toggle_hotkey
-                    .parse()
-                    .unwrap_or(tile.hotkeys.toggle),
-                clipboard_hotkey: new_config
-                    .clipboard_hotkey
-                    .parse()
-                    .unwrap_or(tile.hotkeys.clipboard_hotkey),
-            };
-
-            tile.hotkeys.register();
-
             tile.theme = new_config.theme.to_owned().into();
             tile.config = new_config;
             tile.options = AppIndex::from_apps(new_options);