From: Tony Crisci <tony@dubstepdish.com>
Date: Sat, 20 Sep 2014 21:21:08 +0000 (-0400)
Subject: Do not resend focus on click
X-Git-Tag: 4.9~52
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=8870edc2ca9c187d0358f11c13eb2dc775302dce;p=i3%2Fi3

Do not resend focus on click

Do not set focused_id to XCB_NONE in route click to force resend focus,
in some cases to an already focused window.

Sending focus again on click is not necessary and may cause problems
with certain wine or mono apps. Resending focus makes combo boxes not
work in Office 2010.

This effectively reverts commit 250577da, so in case this commit causes
any problems with Eclipse to resurface, we’ll revert this commit in
favor of 250577da (the Eclipse-related bug fix).
---

diff --git a/src/click.c b/src/click.c
index f501c769..5c36aeb0 100644
--- a/src/click.c
+++ b/src/click.c
@@ -214,7 +214,6 @@ static int route_click(Con *con, xcb_button_press_event_t *event, const bool mod
 
     if (ws != focused_workspace)
         workspace_show(ws);
-    focused_id = XCB_NONE;
 
     /* get the floating con */
     Con *floatingcon = con_inside_floating(con);
diff --git a/src/x.c b/src/x.c
index ff0a2295..ae97ef0d 100644
--- a/src/x.c
+++ b/src/x.c
@@ -15,9 +15,9 @@
 /* Stores the X11 window ID of the currently focused window */
 xcb_window_t focused_id = XCB_NONE;
 
-/* Because 'focused_id' might be reset to force input focus (after click to
- * raise), we separately keep track of the X11 window ID to be able to always
- * tell whether the focused window actually changed. */
+/* Because 'focused_id' might be reset to force input focus, we separately keep
+ * track of the X11 window ID to be able to always tell whether the focused
+ * window actually changed. */
 static xcb_window_t last_focused = XCB_NONE;
 
 /* Stores coordinates to warp mouse pointer to if set */