open.source/espanso
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(engine): prevent other keys from interfering with toggle_key. Fix #815

Browse Source
This commit is contained in:
Federico Terzi 2021-10-24 21:46:09 +02:00
parent 8bab0f5e42
commit 21bc75bc31
1 changed files with 13 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
espanso-engine/src/process/middleware/disable.rs
View File

@ -64,18 +64,24 @@ impl Middleware for DisableMiddleware {
match &event.etype {
EventType::Keyboard(m_event) => {
if is_toggle_key(m_event, &self.options) {
if m_event.status == Status::Released {
let mut last_toggle_press = self.last_toggle_press.borrow_mut();
if let Some(previous_press) = *last_toggle_press {
if previous_press.elapsed() < self.options.toggle_key_maximum_window {
*enabled = !*enabled;
*last_toggle_press = None;
has_status_changed = true;
if is_toggle_key(m_event, &self.options) {
if let Some(previous_press) = *last_toggle_press {
if previous_press.elapsed() < self.options.toggle_key_maximum_window {
*enabled = !*enabled;
*last_toggle_press = None;
has_status_changed = true;
} else {
*last_toggle_press = Some(Instant::now());
}
} else {
*last_toggle_press = Some(Instant::now());
}
} else {
*last_toggle_press = Some(Instant::now());
// If another key is pressed (not the toggle key), we should reset the window
// For more information, see: https://github.com/federico-terzi/espanso/issues/815
*last_toggle_press = None;
}
}
}
@ -115,10 +121,6 @@ impl Middleware for DisableMiddleware {
}
fn is_toggle_key(event: &KeyboardEvent, options: &DisableOptions) -> bool {
if event.status != Status::Released {
return false;
}
if options
.toggle_key
.as_ref()