From 25616c3b9f332eb857b31397fae22a1d7a67aa7d Mon Sep 17 00:00:00 2001
From: Don Stewart <dons@cse.unsw.edu.au>
Date: Sun, 27 May 2007 15:32:11 +0000
Subject: [PATCH] comments on alternative gap movement policies

---
 Operations.hs | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Operations.hs b/Operations.hs
index 32fdea9..e1ee0cd 100644
--- a/Operations.hs
+++ b/Operations.hs
@@ -210,11 +210,14 @@ setTopFocus :: X ()
 setTopFocus = withWorkspace $ maybe (setFocusX =<< asks theRoot) setFocusX . W.peek
 
 -- | Set focus explicitly to window 'w' if it is managed by us, or root.
+-- This happens if X notices we've moved the mouse (and perhaps moved
+-- the mouse to a new screen).
 focus :: Window -> X ()
 focus w = withWorkspace $ \s -> do
     if W.member w s then do modify $ \st -> st { windowset = W.focusWindow w s } -- avoid 'refresh'
                             refresh -- and set gap -- was: setFocusX w
                     else whenX (isRoot w) $ setFocusX w
+                            -- we could refresh here, moving gap too.
     -- XXX a focus change could be caused by switching workspaces in xinerama.
     -- if so, and the gap is in use, the gap should probably follow the
     -- cursor to the new screen.