<div dir="ltr">Hi all,<br>Just wanted to check and see if there were any further issues with this patch.<br><br>Thanks,<br>Arjun<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 19, 2014 at 3:12 PM, Arjun Comar <span dir="ltr"><<a href="mailto:nrujac@gmail.com" target="_blank">nrujac@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">Hi all, <br>Here's an updated patch that switches TabbarLocation with Direction2D.<br><br>Thanks,<br>

Arjun<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 17, 2014 at 6:32 PM, Arjun Comar <span dir="ltr"><<a href="mailto:nrujac@gmail.com" target="_blank">nrujac@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">Hi Daniel,<br>I can confirm that existing configs shouldn't break unless the user has behavior that operates on TabbarLocation (which seems weird, that kind of stuff really belongs in XMonad.Layout.Tabbed). If they *do* have that kind of behavior, they'll need to update their pattern matching to handle the LeftS/RightS cases.<br>



<br>The naming is weird, but it's to prevent conflicts with Either. I was going for a Leftside/Rightside mnemonic. I actually like Adam's idea of just using Direction2D. That should also not break user configs since the interface functions don't ask for Top/Bottom/etc. and instead have separate functions for each task.<br>



<br>Thanks,<br>Arjun<br></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 17, 2014 at 4:16 PM, adam vogt <span dir="ltr"><<a href="mailto:vogt.adam@gmail.com" target="_blank">vogt.adam@gmail.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Daniel and Arjun,<br>
<br>
LeftS and RightS don't take arguments like Left and Right do.<br>
TabbarLocation could be defined in terms of<br>
<a href="http://xmonad.org/xmonad-docs/xmonad-contrib/XMonad-Util-Types.html#t:Direction2D" target="_blank">http://xmonad.org/xmonad-docs/xmonad-contrib/XMonad-Util-Types.html#t:Direction2D</a>,<br>
but it's fine as-is too.<br>
<br>
--<br>
Adam<br>
<div><div>_______________________________________________<br>
xmonad mailing list<br>
<a href="mailto:xmonad@haskell.org" target="_blank">xmonad@haskell.org</a><br>
<a href="http://www.haskell.org/mailman/listinfo/xmonad" target="_blank">http://www.haskell.org/mailman/listinfo/xmonad</a><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>