fix(whkd): bind rwin by default when binding win

This commit ensures that `VKey::RWIN` is registered alongside with
`VKey::LWIN` when specifying a `win` as a modifier.

This is required  because `win-hotkeys` doesn't handle lwin and rwin the
same way that it handles shift, alt and ctrl.

Author: akozlev

core/src/lib.rs

@@ -10,6 +10,33 @@ pub struct Whkdrc {
     pub pause_hook: Option<String>,
 }
 
+impl Whkdrc {
+    pub fn add_rwin_bindings(&mut self) {
+        let mut rwin_bindings = vec![];
+        for binding in &mut self.bindings {
+            if let Some(i) = binding.keys.iter().position(|key| key == "win") {
+                let mut rwin_binding = binding.clone();
+                rwin_binding.keys[i] = "rwin".to_string();
+                rwin_bindings.push(rwin_binding);
+            }
+        }
+        self.bindings.extend(rwin_bindings);
+
+        let mut rwin_app_bindings = vec![];
+        for binding in &mut self.app_bindings {
+            if let Some(i) = binding.0.iter().position(|key| key == "win") {
+                let mut rwin_binding = binding.clone();
+                rwin_binding.0[i] = "rwin".to_string();
+                for app_binding in &mut rwin_binding.1 {
+                    app_binding.keys[i] = "rwin".to_string();
+                }
+                rwin_app_bindings.push(rwin_binding);
+            }
+        }
+        self.app_bindings.extend(rwin_app_bindings);
+    }
+}
+
 #[derive(Debug, Copy, Clone, PartialEq, Eq)]
 pub enum Shell {
     Cmd,

src/main.rs

@@ -37,7 +37,9 @@ lazy_static! {
             },
         );
         home.push("whkdrc");
-        whkd_parser::load(&home).unwrap_or_else(|_| panic!("could not load whkdrc from {home:?}"))
+        let mut parsed = whkd_parser::load(&home).unwrap_or_else(|_| panic!("could not load whkdrc from {home:?}"));
+        parsed.add_rwin_bindings();
+        parsed
     };
     static ref SESSION_STDIN: Mutex<Option<ChildStdin>> = Mutex::new(None);
 }
@@ -173,8 +175,10 @@ fn main() -> Result<()> {
     let whkdrc = cli.config.map_or_else(
         || WHKDRC.clone(),
         |config| {
-            whkd_parser::load(&config)
-                .unwrap_or_else(|_| panic!("could not load whkdrc from {config:?}"))
+            let mut parsed = whkd_parser::load(&config)
+                .unwrap_or_else(|_| panic!("could not load whkdrc from {config:?}"));
+            parsed.add_rwin_bindings();
+            parsed
         },
     );