From ba52dc337860f7dfdd92ecd552277ef9ec00edfa Mon Sep 17 00:00:00 2001 From: LGUG2Z Date: Sat, 4 Jan 2025 10:17:31 -0800 Subject: [PATCH] fix(wm): add uncloak as a notif override event If a user triggers the workspace reconciliator by clicking on an app in the start bar or via alt-tab, a notification should be sent to subscribers such as komorebi-bar so that the focused workspace can be updated. The various komorebi reconciliators and manager modules don't emit events to subscribers themselves (yet?), so for now we can pass on the uncloak event. Maybe we can look into expanding the Notification enum in the future. fix #1211 --- komorebi/src/lib.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/komorebi/src/lib.rs b/komorebi/src/lib.rs index 42e39bad..38a83c50 100644 --- a/komorebi/src/lib.rs +++ b/komorebi/src/lib.rs @@ -298,6 +298,7 @@ pub fn notify_subscribers(notification: Notification, state_has_been_modified: b | NotificationEvent::Socket(SocketMessage::ReloadStaticConfiguration(_)) | NotificationEvent::WindowManager(WindowManagerEvent::TitleUpdate(_, _)) | NotificationEvent::WindowManager(WindowManagerEvent::Show(_, _)) + | NotificationEvent::WindowManager(WindowManagerEvent::Uncloak(_, _)) ); let notification = &serde_json::to_string(¬ification)?;