On Wed, Jun 08, 2011 at 11:59:36PM -0400, Scott Jaderholm wrote:
> When I make a placement rule that isn't supposed to focus the window (nil as
> third field in dumped rules) it only works (doesn't focus the window) when
> there's already a window in the destination frame. Anyone else have this
> problem? is it a bug? Or is the scope of the raise option really only
> raising not focusing?

Hi Scott,

Looks like a bug.  The issue is that group-add-window raises and
focuses the window when it's the only one in its frame.  Attached is a
patch to make it only raise the window in that case, but I'm not sure
if this will break anything else.

Ben
>From f933810b7eaa4f46b43d6b31734903dab4358e26 Mon Sep 17 00:00:00 2001
From: Ben Spencer <dangerous....@gmail.com>
Date: Tue, 12 Jul 2011 13:20:19 +0100
Subject: [PATCH] Only focus window in group-add-window if raise is specified.

---
 tile-group.lisp |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tile-group.lisp b/tile-group.lisp
index 72d8d73..29cdb2b 100644
--- a/tile-group.lisp
+++ b/tile-group.lisp
@@ -72,7 +72,7 @@
   (sync-frame-windows group (window-frame window))
   ;; maybe show the window in its new frame
   (when (null (frame-window (window-frame window)))
-    (really-raise-window window)))
+    (frame-raise-window group (window-frame window) window raise)))
 
 (defmethod group-current-window ((group tile-group))
   (frame-window (tile-group-current-frame group)))
-- 
1.7.5.4

_______________________________________________
Stumpwm-devel mailing list
Stumpwm-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/stumpwm-devel

Reply via email to