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

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 :

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

...

This is such a small change that I think that it makes more sense to

On Tue, Mar 3, 2015 at 4:09 PM, Anton Pirogov wrote: 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@haskell.org http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad