[xmonad] one more layout

Roman Cheplyaka roma at ro-che.info
Tue Apr 14 06:38:10 EDT 2009


* Konstantin Sobolev <konstantin.sobolev at gmail.com> [2009-04-03 17:00:36-0400]
> Hello,
> 
> Could somebody review my attempt to write a layout and tell if it's
> worth including in contrib?

Sure, it seems to be useful.

There are several wishes to your code:
1. All non-trivial top-level functions should be documented (such as
   swap, forwardToFocused etc.)

2. If you use some function from a different module (like
   'differentiate' from Combo) you shouldn't copy-paste its code;
   instead prepare a separate patch which adds that function to export
   list and explain where you use that function in the patch
   description. Later that function may be also moved to some module in
   the Util section.
   
   Again, those functions (differentiate and broadcastPrivate from
   Combo) are not documented there. Could you please write documentation
   for them if you use them?

3. You shouldn't advise to import XMonad.Util.WindowProperties; instead
   you should re-export needed things (or even the whole module) from
   your module. See XMonad.Layout.IM for example.

Sorry for a late response and thanks for your work!

-- 
Roman I. Cheplyaka (aka Feuerbach @ IRC)
http://ro-che.info/docs/xmonad.hs


More information about the xmonad mailing list