From b1d69a552cdcc7370c4befd42ef2111c8b367c46 Mon Sep 17 00:00:00 2001 From: srdusr <99972264+srdusr@users.noreply.github.com> Date: Tue, 11 Feb 2025 14:00:00 +0200 Subject: Add temporary diagnostics for two live-reproduced bugs 1. A popup's xdg_popup.grab (Firefox's own right-click menu, concretely) receiving zero pointer input at all - no hover highlight, no click effect, not even dismiss-on-miss - logs whether grab_popup actually succeeds, since a silent failure there would explain exactly this. 2. srd dispatch activate_workspace returning {"ok":true} without ever changing the current workspace, confirmed via a raw socket request bypassing the CLI entirely. switch_workspace's own logic reads correct; logs its actual inputs/state to find out why the real process disagrees with it. Remove once both are resolved. --- crates/core/src/manager/workspaces.rs | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) (limited to 'crates/core/src') diff --git a/crates/core/src/manager/workspaces.rs b/crates/core/src/manager/workspaces.rs index 6ffb21b..ddc8341 100644 --- a/crates/core/src/manager/workspaces.rs +++ b/crates/core/src/manager/workspaces.rs @@ -50,7 +50,14 @@ impl WindowManager { /// doesn't need its own separate bookkeeping. pub fn switch_workspace(&mut self, id: WorkspaceId) { let target = if self.auto_back_and_forth && id == self.current_workspace { self.previous_workspace } else { id }; - if self.workspaces.iter().any(|w| w.id == target) && target != self.current_workspace { + let exists = self.workspaces.iter().any(|w| w.id == target); + log::warn!( + "SWITCH-WS-DIAG requested_id={id} auto_back_and_forth={} current={} previous={} target={target} exists={exists}", + self.auto_back_and_forth, + self.current_workspace, + self.previous_workspace + ); + if exists && target != self.current_workspace { self.previous_workspace = self.current_workspace; self.current_workspace = target; } -- cgit v1.2.3