From 25b84d6386701c79e6fb389c1d0ce843a48f0243 Mon Sep 17 00:00:00 2001 From: Devin Mullins Date: Mon, 11 Aug 2008 05:31:37 +0200 Subject: simplify WindowBringer code, and change greedyView to focusWindow darcs-hash:20080811033137-78224-6fb0ea21833cf4400fce90981b07826d26162451.gz --- XMonad/Actions/WindowBringer.hs | 31 ++++++++++++++++--------------- XMonad/Prompt/Window.hs | 10 ++++------ 2 files changed, 20 insertions(+), 21 deletions(-) diff --git a/XMonad/Actions/WindowBringer.hs b/XMonad/Actions/WindowBringer.hs index cd31de8..83ba481 100644 --- a/XMonad/Actions/WindowBringer.hs +++ b/XMonad/Actions/WindowBringer.hs @@ -17,7 +17,8 @@ module XMonad.Actions.WindowBringer ( -- * Usage -- $usage - gotoMenu, bringMenu, windowMapWith + gotoMenu, bringMenu, windowMap, + bringWindow ) where import Data.Char (toLower) @@ -47,29 +48,29 @@ import XMonad.Util.NamedWindows (getName) -- | Pops open a dmenu with window titles. Choose one, and you will be -- taken to the corresponding workspace. gotoMenu :: X () -gotoMenu = workspaceMap >>= actionMenu (windows . W.greedyView) - where workspaceMap = windowMapWith (W.tag . fst) +gotoMenu = actionMenu W.focusWindow -- | Pops open a dmenu with window titles. Choose one, and it will be -- dragged, kicking and screaming, into your current workspace. bringMenu :: X () -bringMenu = windowMap >>= actionMenu (windows . bringWindow) - where windowMap = windowMapWith snd - bringWindow w ws = W.shiftWin (W.tag . W.workspace . W.current $ ws) w ws +bringMenu = actionMenu bringWindow --- | Calls dmenuMap to grab the appropriate element from the Map, and hands it --- off to action if found. -actionMenu :: (a -> X ()) -> M.Map String a -> X () -actionMenu action windowMap = dmenuMap windowMap >>= flip X.whenJust action +-- | Brings the specified window into the current workspace. +bringWindow :: Window -> X.WindowSet -> X.WindowSet +bringWindow w ws = W.shiftWin (W.tag . W.workspace . W.current $ ws) w ws --- | Generates a Map from window name to \. For --- use with dmenuMap. -windowMapWith :: ((X.WindowSpace, Window) -> a) -> X (M.Map String a) -windowMapWith value = do -- TODO: extract the pure, creamy center. +-- | Calls dmenuMap to grab the appropriate Window, and hands it off to action +-- if found. +actionMenu :: (Window -> X.WindowSet -> X.WindowSet) -> X() +actionMenu action = windowMap >>= dmenuMap >>= flip X.whenJust (windows . action) + +-- | A map from window names to Windows. +windowMap :: X (M.Map String Window) +windowMap = do ws <- gets X.windowset M.fromList `fmap` concat `fmap` mapM keyValuePairs (W.workspaces ws) where keyValuePairs ws = mapM (keyValuePair ws) $ W.integrate' (W.stack ws) - keyValuePair ws w = flip (,) (value (ws, w)) `fmap` decorateName ws w + keyValuePair ws w = flip (,) w `fmap` decorateName ws w -- | Returns the window name as will be listed in dmenu. -- Lowercased, for your convenience (since dmenu is case-sensitive). diff --git a/XMonad/Prompt/Window.hs b/XMonad/Prompt/Window.hs index 65d4009..c45417e 100644 --- a/XMonad/Prompt/Window.hs +++ b/XMonad/Prompt/Window.hs @@ -73,16 +73,14 @@ windowPromptBring c = doPrompt Bring c doPrompt :: WindowPrompt -> XPConfig -> X () doPrompt t c = do a <- case t of - Goto -> return . gotoAction =<< windowMapWith (W.tag . fst) - Bring -> return . bringAction =<< windowMapWith snd - wm <- windowMapWith id + Goto -> fmap gotoAction windowMap + Bring -> fmap bringAction windowMap + wm <- windowMap mkXPrompt t c (compList wm) a where - winAction a m = flip whenJust (windows . a) . flip M.lookup m - gotoAction = winAction W.greedyView + gotoAction = winAction W.focusWindow bringAction = winAction bringWindow - bringWindow w ws = W.shiftWin (W.tag . W.workspace . W.current $ ws) w ws compList m s = return . filter (isPrefixOf s) . map fst . M.toList $ m -- cgit v1.2.3