<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 15, 2014 at 11:50 AM, Brandon Allbery <span dir="ltr"><<a href="mailto:allbery.b@gmail.com" target="_blank">allbery.b@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Fri, Dec 12, 2014 at 12:10 PM, Norbert Zeh <span dir="ltr"><<a href="mailto:nzeh@cs.dal.ca" target="_blank">nzeh@cs.dal.ca</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">... and thought I'd share.  Patch is attached.<br></div></blockquote><div><br></div></span><div>A few nits:</div><div><br></div><div>- the comment in XMonad.Layout.Dwindle identifies it as XMonad.Layout.Spiral</div><div><br></div><div>- 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.</div></div></div></div></blockquote><div><br></div><div>Thanks for the feedback.  Ended up in my spam.  I'll update that and resend soon.</div><div><br></div><div>Cheers,</div><div>Norbert </div></div></div></div>