sillysloft / fluxbox

Fluxbox Window Manager (Mirror)
http://fluxbox.org/news/
Other
0 stars 1 forks source link

focus isn't adjusted when :Lower is bound to a mouse button #764

Open sillysloft opened 17 years ago

sillysloft commented 17 years ago

I wanted to lower a window with control+right mouse, so I added this to my keys file: Control Mouse3 :Lower

While it does lower the window, Fluxbox doesn't adjust the focus if the pointer happens to land on another window when using focus follows mouse. Focus does move if it's bound to a keyboard-only action (control+l or something).

I'm currently using my older patch, which doesn't interfere with any focus change.

Not sure if this is a similar issue to which Openbox has a workaround, which rechecks the client under the pointer. Versions 3.3.992-994 reverted this in favor of having the user go with Click or Release events instead of Press. I didn't see a way to select press, click, or release with the keys file or even limit this action to a window, but I assume it's all press...

Reported by: lgn

sillysloft commented 17 years ago

Logged In: YES user_id=1538990 Originator: NO

Actually, it's the same as https://sourceforge.net/tracker/index.php?func=detail&aid=1690857&group\_id=35398&atid=413960 . We're ignoring the EnterNotify event because it has NotifyUngrab set. To be honest, I'm not completely sure why it does, but it does. We're still working on the best solution to the problem, and this bug makes things a little more complicated.

No, there is no support yet for limiting to a window. That's planned for after 1.0. There's also no support for specifying buttonpress vs. buttonrelease, and I don't really plan on it. However, it would be utterly trivial to add, so I'm sure it will happen eventually. Anyway, that wouldn't affect this, since the EnterNotify event is the problem, not the click. (limiting to the window would work around this problem, but it wouldn't fix this problem)

Original comment by: mark-t

sillysloft commented 13 years ago

I think the window focus should always be re-evaluated after a :Lower event. If there are multiple maximized windows, then after lowering the one at the bottom has focus, which makes no sense.

Original comment by: *anonymous

sillysloft commented 8 years ago

this is 1012, patch pending as https://github.com/luebking/fluxbox/commit/a512f91727c3557fed0323976a58fb0aeb7add5f

Original comment by: baghira-style

sillysloft commented 8 years ago

Original comment by: akir