[xmonad] [xmonad-contrib] Patch to add messages IncSpacing/SetSpacing to the Spacing modifier for dynamic adjustment

[Anton Pirogov]

Hi Adam,

thank you for your suggestions! Indeed I didn't need those instances and I
changed the message to your more flexible idea!

I don't know what is the right way to do it here and with darcs in general,
I used "darcs obliterate --last 1" and recorded a new patch. Here is the
updated patch, I hope it is ok now.

Regards,
Anton

2015-03-04 1:03 GMT+01:00 adam vogt <vogt.adam at gmail.com>:

> Hi Anton,
>
> Do you actually need the Read/Show instances? I think it's better to
> put a function in the message:
>
> data ModifySpacing = ModifySpacing (Int -> Int)
>
> And possibly export these convenience functions:
>
> setSpacing n = sendMessage (ModifySpacing (\_ -> n))
> incSpacing n = sendMessage (ModifySpacing (+n))
>
> Regards,
> Adam
>
> On Tue, Mar 3, 2015 at 4:09 PM, Anton Pirogov <anton.pirogov at gmail.com>
> wrote:
> > This is such a small change that I think that it makes more sense to
> patch
> > it, instead of rolling my own. It just adds the message, nothing more,
> so it
> > does not break anything.
> >
> > Thanks!
> >
> > Best regards,
> > Anton Pirogov
> >
> > _______________________________________________
> > xmonad mailing list
> > xmonad at haskell.org
> > http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/xmonad/attachments/20150304/04dcfa2c/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: spacingWithMsg2.dpatch
Type: application/octet-stream
Size: 30284 bytes
Desc: not available
URL: <http://mail.haskell.org/pipermail/xmonad/attachments/20150304/04dcfa2c/attachment-0001.obj>