xmonad.hs: Fix nextLayoutIncluding changing layout on all workspaces.

diff --git a/xmonad.hs b/xmonad.hs
index d8f4d2dd9ad3859334387804ef4d171814f520cb..04257cb6dab0c809d9731b5de59f6daf77c19ad1 100644 (file)
--- a/xmonad.hs
+++ b/xmonad.hs
@@ -90,17 +90,28 @@ myManageHook = composeOne
     , return True -?> doF W.focusDown
     ]
 
--- Switch to next layout with descriptions in desc, other layouts are skipped.
--- This allows switching to some layouts with mappings but excluding them from
+-- Switch to next layout, but skip all layouts not in layouts argument. This
+-- allows switching to some layouts with mappings but excluding them from
 -- meta-space (which gets mapped to this function). Thanks to aavogt in
--- #xmonad on Freenode for this function (2010-06-13 12:45 CEST), modified to
--- take list of layouts to switch to, not layouts to exclude.
-nextLayoutIncluding descs =
-    do broadcastMessage NextLayout
-       d <- gets (description . W.layout . W.workspace . W.current . windowset)
-       if d `elem` descs
-           then refresh
-           else nextLayoutIncluding descs
+-- #xmonad on Freenode for this function (2011-06-13 12:45 CEST) and
+-- rootzlevel in #xmonad on Freenode for fixes (2011-06-13 15:20 CEST),
+-- modified to take list of layouts to switch to, not layouts to exclude.
+nextLayoutIncluding :: [String] -> X ()
+nextLayoutIncluding layouts = do
+    cws <- gets (W.workspace . W.current . windowset)
+    sendMessageWithNoRefresh NextLayout cws
+    nextLayoutIncluding' layouts
+
+nextLayoutIncluding' :: [String] -> X ()
+nextLayoutIncluding' layouts = do
+    cws <- gets (W.workspace . W.current . windowset)
+    if not $ (description $ W.layout cws) `elem` layouts
+        -- Skip over excluded layouts.
+        then do
+            sendMessageWithNoRefresh NextLayout cws
+            nextLayoutIncluding' layouts
+        -- Found allowed layout, show it.
+        else refresh
 
 myKeys = [
         -- Switch to next layout, but only use the listed layouts.