]> git.sur5r.net Git - i3/i3/blobdiff - src/handlers.c
Check output crossing on ENTER_NOTIFY to dockarea. (#2477)
[i3/i3] / src / handlers.c
index 50e218c80edc3734c6808058a4015072ce24d4a7..8a84f33500d2d41675f32b4c7a8f17268a5d6158 100644 (file)
@@ -148,18 +148,15 @@ static void handle_enter_notify(xcb_enter_notify_event_t *event) {
         enter_child = true;
     }
 
-    /* If not, then the user moved their cursor to the root window. In that case, we adjust c_ws */
-    if (con == NULL) {
+    /* If we cannot find the container, the user moved their cursor to the root
+     * window. In this case and if they used it to a dock, we need to focus the
+     * workspace on the correct output. */
+    if (con == NULL || con->parent->type == CT_DOCKAREA) {
         DLOG("Getting screen at %d x %d\n", event->root_x, event->root_y);
         check_crossing_screen_boundary(event->root_x, event->root_y);
         return;
     }
 
-    if (con->parent->type == CT_DOCKAREA) {
-        DLOG("Ignoring, this is a dock client\n");
-        return;
-    }
-
     /* see if the user entered the window on a certain window decoration */
     layout_t layout = (enter_child ? con->parent->layout : con->layout);
     if (layout == L_DEFAULT) {
@@ -172,16 +169,6 @@ static void handle_enter_notify(xcb_enter_notify_event_t *event) {
         }
     }
 
-#if 0
-    if (client->workspace != c_ws && client->workspace->output == c_ws->output) {
-            /* This can happen when a client gets assigned to a different workspace than
-             * the current one (see src/mainx.c:reparent_window). Shortly after it was created,
-             * an enter_notify will follow. */
-            DLOG("enter_notify for a client on a different workspace but the same screen, ignoring\n");
-            return 1;
-    }
-#endif
-
     if (config.disable_focus_follows_mouse)
         return;
 
@@ -421,22 +408,6 @@ static void handle_configure_request(xcb_configure_request_event_t *event) {
 
     return;
 }
-#if 0
-
-/*
- * Configuration notifies are only handled because we need to set up ignore for
- * the following enter notify events.
- *
- */
-int handle_configure_event(void *prophs, xcb_connection_t *conn, xcb_configure_notify_event_t *event) {
-    DLOG("configure_event, sequence %d\n", event->sequence);
-        /* We ignore this sequence twice because events for child and frame should be ignored */
-        add_ignore_event(event->sequence);
-        add_ignore_event(event->sequence);
-
-        return 1;
-}
-#endif
 
 /*
  * Gets triggered upon a RandR screen change event, that is when the user
@@ -503,8 +474,10 @@ static void handle_unmap_notify_event(xcb_unmap_notify_event_t *event) {
         goto ignore_end;
     }
 
-    /* Since we close the container, we need to unset _NET_WM_DESKTOP according to the spec. */
+    /* Since we close the container, we need to unset _NET_WM_DESKTOP and
+     * _NET_WM_STATE according to the spec. */
     xcb_delete_property(conn, event->window, A__NET_WM_DESKTOP);
+    xcb_delete_property(conn, event->window, A__NET_WM_STATE);
 
     tree_close_internal(con, DONT_KILL_WINDOW, false, false);
     tree_render();
@@ -628,23 +601,6 @@ static bool handle_windowrole_change(void *data, xcb_connection_t *conn, uint8_t
     return true;
 }
 
-#if 0
-/*
- * Updates the client’s WM_CLASS property
- *
- */
-static int handle_windowclass_change(void *data, xcb_connection_t *conn, uint8_t state,
-                             xcb_window_t window, xcb_atom_t atom, xcb_get_property_reply_t *prop) {
-    Con *con;
-    if ((con = con_by_window_id(window)) == NULL || con->window == NULL)
-        return 1;
-
-    window_update_class(con->window, prop, false);
-
-    return 0;
-}
-#endif
-
 /*
  * Expose event means we should redraw our windows (= title bar)
  *
@@ -881,7 +837,7 @@ static void handle_client_message(xcb_client_message_event_t *event) {
                 return;
             }
 
-            con_move_to_workspace(con, ws, false, false, true);
+            con_move_to_workspace(con, ws, true, false, false);
         }
 
         tree_render();
@@ -1212,6 +1168,38 @@ static bool handle_class_change(void *data, xcb_connection_t *conn, uint8_t stat
     return true;
 }
 
+/*
+ * Handles the _MOTIF_WM_HINTS property of specifing window deocration settings.
+ *
+ */
+static bool handle_motif_hints_change(void *data, xcb_connection_t *conn, uint8_t state, xcb_window_t window,
+                                      xcb_atom_t name, xcb_get_property_reply_t *prop) {
+    Con *con;
+    if ((con = con_by_window_id(window)) == NULL || con->window == NULL)
+        return false;
+
+    if (prop == NULL) {
+        prop = xcb_get_property_reply(conn, xcb_get_property_unchecked(conn,
+                                                                       false, window, A__MOTIF_WM_HINTS, XCB_GET_PROPERTY_TYPE_ANY, 0, 5 * sizeof(uint64_t)),
+                                      NULL);
+
+        if (prop == NULL)
+            return false;
+    }
+
+    border_style_t motif_border_style;
+    window_update_motif_hints(con->window, prop, &motif_border_style);
+
+    if (motif_border_style != con->border_style && motif_border_style != BS_NORMAL) {
+        DLOG("Update border style of con %p to %d\n", con, motif_border_style);
+        con_set_border_style(con, motif_border_style, con->current_border_width);
+
+        x_push_changes(croot);
+    }
+
+    return true;
+}
+
 /*
  * Handles the _NET_WM_STRUT_PARTIAL property for allocating space for dock clients.
  *
@@ -1313,7 +1301,8 @@ static struct property_handler_t property_handlers[] = {
     {0, 128, handle_windowrole_change},
     {0, 128, handle_class_change},
     {0, UINT_MAX, handle_strut_partial_change},
-    {0, UINT_MAX, handle_window_type}};
+    {0, UINT_MAX, handle_window_type},
+    {0, 5 * sizeof(uint64_t), handle_motif_hints_change}};
 #define NUM_HANDLERS (sizeof(property_handlers) / sizeof(struct property_handler_t))
 
 /*
@@ -1334,6 +1323,7 @@ void property_handlers_init(void) {
     property_handlers[7].atom = XCB_ATOM_WM_CLASS;
     property_handlers[8].atom = A__NET_WM_STRUT_PARTIAL;
     property_handlers[9].atom = A__NET_WM_WINDOW_TYPE;
+    property_handlers[10].atom = A__MOTIF_WM_HINTS;
 }
 
 static void property_notify(uint8_t state, xcb_window_t window, xcb_atom_t atom) {