Skip to content

Commit

Permalink
fix(wm): add uncloak as a notif override event
Browse files Browse the repository at this point in the history
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
  • Loading branch information
LGUG2Z committed Jan 5, 2025
1 parent 44716fd commit ba52dc3
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions komorebi/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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(&notification)?;
Expand Down

0 comments on commit ba52dc3

Please sign in to comment.