Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-05-12 Thread Rob
On 11 May 2011 12:39, Connor Lane Smith wrote: > Hey Rob, > > I think this was caused by the given '-m' not being present. I've > applied a patch which should fix this, since it falls back to the > mouse if the '-m' isn't there. Could you confirm whether it works? > > If it doesn't it's a weird co

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-05-11 Thread Connor Lane Smith
Hey Rob, On 11 May 2011 12:01, Rob wrote: > Sorry to thread-dig here, but I've had trouble with dmenu recently and > traced it to this patch. The XCreateWindow call would fail because mw > would be zero. I'm unsure why, but this problem only happened when > Firefox or OpenOffice were focussed. An

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-05-11 Thread Rob
Hi, Sorry to thread-dig here, but I've had trouble with dmenu recently and traced it to this patch. The XCreateWindow call would fail because mw would be zero. I'm unsure why, but this problem only happened when Firefox or OpenOffice were focussed. Anyway, I've sorted it, the patch caused dmenu to

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-03-12 Thread Axel Wagner
Hi, Excerpts from Connor Lane Smith's message of 2011-03-12 23:49:22 +0100: > I feel the patch is a little elaborate for inclusion in trunk, but I > might consider making focus default -- it just makes the code a bit > messier. No, problem, the command-line stuff was actually only there as a preem

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-03-12 Thread Connor Lane Smith
Hey, On 11 March 2011 20:11, Axel Wagner wrote: > With the attached patch, -m can (additionaly to taking a number) also > takes „focus“ and „mouse“ as options. The latter represents the current > default behaviour, the former uses the center of the window which > currently has the input-focus to

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-03-12 Thread Moritz Wilhelmy
Hi, On Sat, Mar 12, 2011 at 11:20:24PM +0100, Axel Wagner wrote: > First, I use dmenu in conjunction with i3, so I can't use that patch, > even if I wanted to (though it would be trivially to implement, I think) ;) > Second, as I said, I hate mouse-follows-focus, imho the mouse should > move, when

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-03-12 Thread Axel Wagner
Hi, Excerpts from Moritz Wilhelmy's message of 2011-03-12 18:47:56 +0100: > For your information, there is a "warp" patch on the wiki that warps the mouse > pointer to the active window on focus change. First, I use dmenu in conjunction with i3, so I can't use that patch, even if I wanted to (tho

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-03-12 Thread Moritz Wilhelmy
Hi, For your information, there is a "warp" patch on the wiki that warps the mouse pointer to the active window on focus change. Best regards, Moritz

Re: [dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-03-12 Thread Pascal Wittmann
On 03/11/2011 09:11 PM, Axel Wagner wrote: > Moin, > > if no -m option is given to dmenu, it infers it's xinerama-screen from > the mouse-position. On heavily keyboard-driven window-managers, the > mouse-position differs heavily from where the attention is sometimes (I > personally hate mouse-foll

[dev] [PATCH] dmenu: Add commandline-options to determine Xinerama-screen

2011-03-11 Thread Axel Wagner
Moin, if no -m option is given to dmenu, it infers it's xinerama-screen from the mouse-position. On heavily keyboard-driven window-managers, the mouse-position differs heavily from where the attention is sometimes (I personally hate mouse-follows-focus, so when I change the focus via keyboard to a