* On Thursday, October 22 2009, Daniel Schoepe wrote: [...]
I think that the benefits of this patch outweigh the breakage since the current behavior is somewhat broken in my opinion.
Applied, thanks!
I would like to avoid config breakage, even if it is only for darcs-users (since the ewmhdesktops changed from a layout to event hook from 0.8). We can accomplish the same thing by making a top-level, non-exported IORef Bool (as is already done in other modules such as UrgencyHook), and then checking that one if we need to set the X atoms. Once Daniel's extensible state patch gets applied in core, it will be simple to replace the IORef with the use of that feature. Attached is a patch that makes these changes. I think it should be applied before the 0.9 freeze, but I'm not going to do that because there were some objections in #xmonad such as: - IORefs are evil - it only applies to darcs version users, and these are supposed to handle these things -- Adam