Re: [xmonad] Added a new simple layout

Hi Norbert, Would you be willing to change the license to the one contrib uses? Otherwise I suppose you should to submit the module to this (unaffiliated) project: https://hackage.haskell.org/package/xmonad-contrib-gpl Also, there's a XMonad.Util.Types.Direction2D that also defines L R U D constructors. It would be better to import and use that one instead of data Direction. Thanks, Adam On Sat, Mar 14, 2015 at 11:54 AM, Norbert Zeh wrote:

On Tue, Mar 3, 2015 at 3:39 PM Norbert Zeh wrote:

...

On Mon, Dec 15, 2014 at 11:50 AM, Brandon Allbery wrote:

...

On Fri, Dec 12, 2014 at 12:10 PM, Norbert Zeh wrote:

...

... and thought I'd share. Patch is attached.

A few nits:

- the comment in XMonad.Layout.Dwindle identifies it as XMonad.Layout.Spiral

- consider renaming Rotation, as anyone using your module must also import XMonad hiding (Rotation). At the very least, you should document the requirement if you're going to keep it, since you export it. Possible alternate name: Chirality.

Updated patch is attached. Let me know if anything else needs to be fixed.

Cheers, Norbert

_______________________________________________ xmonad mailing list xmonad@haskell.org http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad