]> git.sur5r.net Git - i3/i3/blobdiff - src/ewmh.c
Bugfix: Setup the _NET_SUPPORTING_WM_CHECK atom in a standards-compliant way
[i3/i3] / src / ewmh.c
index 6bfa3096ef25c777bf5f8b6d22961d8768022037..a11d46583d51689df8fadccd6fa4b2321873d651 100644 (file)
@@ -1,25 +1,13 @@
 /*
- * vim:ts=8:expandtab
+ * vim:ts=4:sw=4:expandtab
  *
  * i3 - an improved dynamic tiling window manager
+ * © 2009-2011 Michael Stapelberg and contributors (see also: LICENSE)
  *
- * © 2009 Michael Stapelberg and contributors
- *
- * See file LICENSE for license information.
- *
- * ewmh.c: Functions to get/set certain EWMH properties easily.
+ * ewmh.c: Get/set certain EWMH properties easily.
  *
  */
-#include <stdint.h>
-#include <string.h>
-#include <stdlib.h>
-
-#include "data.h"
-#include "table.h"
-#include "i3.h"
-#include "xcb.h"
-#include "util.h"
-#include "log.h"
+#include "all.h"
 
 /*
  * Updates _NET_CURRENT_DESKTOP with the current desktop number.
  *
  */
 void ewmh_update_current_desktop() {
-        uint32_t current_desktop = c_ws->num;
-        xcb_change_property(global_conn, XCB_PROP_MODE_REPLACE, root,
-                            atoms[_NET_CURRENT_DESKTOP], CARDINAL, 32, 1,
-                            &current_desktop);
+    Con *focused_ws = con_get_workspace(focused);
+    Con *output;
+    uint32_t idx = 0;
+    /* We count to get the index of this workspace because named workspaces
+     * don’t have the ->num property */
+    TAILQ_FOREACH(output, &(croot->nodes_head), nodes) {
+        Con *ws;
+        TAILQ_FOREACH(ws, &(output_get_content(output)->nodes_head), nodes) {
+            if (ws == focused_ws) {
+                xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root,
+                        A__NET_CURRENT_DESKTOP, XCB_ATOM_CARDINAL, 32, 1, &idx);
+                return;
+            }
+            ++idx;
+        }
+    }
 }
 
 /*
@@ -43,61 +43,124 @@ void ewmh_update_current_desktop() {
  *
  */
 void ewmh_update_active_window(xcb_window_t window) {
-        xcb_change_property(global_conn, XCB_PROP_MODE_REPLACE, root,
-                            atoms[_NET_ACTIVE_WINDOW], WINDOW, 32, 1, &window);
+    xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root,
+            A__NET_ACTIVE_WINDOW, XCB_ATOM_WINDOW, 32, 1, &window);
 }
 
 /*
  * Updates the workarea for each desktop.
  *
+ * This function is not called at the moment due to:
+ * http://bugs.i3wm.org/539
+ * http://bugs.i3wm.org/301
+ *
  * EWMH: Contains a geometry for each desktop. These geometries specify an area
  * that is completely contained within the viewport. Work area SHOULD be used by
  * desktop applications to place desktop icons appropriately.
  *
  */
 void ewmh_update_workarea() {
-        Workspace *ws;
-        int num_workspaces = 0, count = 0;
-        Rect last_rect = {0, 0, 0, 0};
+    int num_workspaces = 0, count = 0;
+    Rect last_rect = {0, 0, 0, 0};
+    Con *output;
 
-        /* Get the number of workspaces */
-        TAILQ_FOREACH(ws, workspaces, workspaces) {
-                /* Check if we need to initialize last_rect. The case that the
-                 * first workspace is all-zero may happen when the user
-                 * assigned workspace 2 for his first screen, for example. Thus
-                 * we need an initialized last_rect in the very first run of
-                 * the following loop. */
-                if (last_rect.width == 0 && last_rect.height == 0 &&
+    TAILQ_FOREACH(output, &(croot->nodes_head), nodes) {
+        Con *ws;
+        TAILQ_FOREACH(ws, &(output_get_content(output)->nodes_head), nodes) {
+            /* Check if we need to initialize last_rect. The case that the
+             * first workspace is all-zero may happen when the user
+             * assigned workspace 2 for his first screen, for example. Thus
+             * we need an initialized last_rect in the very first run of
+             * the following loop. */
+            if (last_rect.width == 0 && last_rect.height == 0 &&
                     ws->rect.width != 0 && ws->rect.height != 0) {
-                        memcpy(&last_rect, &(ws->rect), sizeof(Rect));
-                }
-                num_workspaces++;
+                memcpy(&last_rect, &(ws->rect), sizeof(Rect));
+            }
+            num_workspaces++;
         }
+    }
 
-        DLOG("Got %d workspaces\n", num_workspaces);
-        uint8_t *workarea = smalloc(sizeof(Rect) * num_workspaces);
-        TAILQ_FOREACH(ws, workspaces, workspaces) {
-                DLOG("storing %d: %dx%d with %d x %d\n", count, ws->rect.x,
-                     ws->rect.y, ws->rect.width, ws->rect.height);
-                /* If a workspace is not yet initialized and thus its
-                 * dimensions are zero, we will instead put the dimensions
-                 * of the last workspace in the list. For example firefox
-                 * intersects all workspaces and does not cope so well with
-                 * an all-zero workspace. */
-                if (ws->rect.width == 0 || ws->rect.height == 0) {
-                        DLOG("re-using last_rect (%dx%d, %d, %d)\n",
-                             last_rect.x, last_rect.y, last_rect.width,
-                             last_rect.height);
-                        memcpy(workarea + (sizeof(Rect) * count++), &last_rect, sizeof(Rect));
-                        continue;
-                }
-                memcpy(workarea + (sizeof(Rect) * count++), &(ws->rect), sizeof(Rect));
-                memcpy(&last_rect, &(ws->rect), sizeof(Rect));
+    DLOG("Got %d workspaces\n", num_workspaces);
+    uint8_t *workarea = smalloc(sizeof(Rect) * num_workspaces);
+    TAILQ_FOREACH(output, &(croot->nodes_head), nodes) {
+        Con *ws;
+        TAILQ_FOREACH(ws, &(output_get_content(output)->nodes_head), nodes) {
+            DLOG("storing %d: %dx%d with %d x %d\n", count, ws->rect.x,
+                 ws->rect.y, ws->rect.width, ws->rect.height);
+            /* If a workspace is not yet initialized and thus its
+             * dimensions are zero, we will instead put the dimensions
+             * of the last workspace in the list. For example firefox
+             * intersects all workspaces and does not cope so well with
+             * an all-zero workspace. */
+            if (ws->rect.width == 0 || ws->rect.height == 0) {
+                DLOG("re-using last_rect (%dx%d, %d, %d)\n",
+                     last_rect.x, last_rect.y, last_rect.width,
+                     last_rect.height);
+                memcpy(workarea + (sizeof(Rect) * count++), &last_rect, sizeof(Rect));
+                continue;
+            }
+            memcpy(workarea + (sizeof(Rect) * count++), &(ws->rect), sizeof(Rect));
+            memcpy(&last_rect, &(ws->rect), sizeof(Rect));
         }
-        xcb_change_property(global_conn, XCB_PROP_MODE_REPLACE, root,
-                            atoms[_NET_WORKAREA], CARDINAL, 32,
-                            num_workspaces * (sizeof(Rect) / sizeof(uint32_t)),
-                            workarea);
-        free(workarea);
-        xcb_flush(global_conn);
+    }
+    xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root,
+            A__NET_WORKAREA, XCB_ATOM_CARDINAL, 32,
+            num_workspaces * (sizeof(Rect) / sizeof(uint32_t)),
+            workarea);
+    free(workarea);
+    xcb_flush(conn);
+}
+
+/*
+ * Updates the _NET_CLIENT_LIST_STACKING hint.
+ *
+ */
+void ewmh_update_client_list_stacking(xcb_window_t *stack, int num_windows) {
+    xcb_change_property(
+        conn,
+        XCB_PROP_MODE_REPLACE,
+        root,
+        A__NET_CLIENT_LIST_STACKING,
+        XCB_ATOM_WINDOW,
+        32,
+        num_windows,
+        stack);
+}
+
+/*
+ * Set up the EWMH hints on the root window.
+ *
+ */
+void ewmh_setup_hints() {
+    xcb_atom_t supported_atoms[] = {
+#define xmacro(atom) A_ ## atom,
+#include "atoms.xmacro"
+#undef xmacro
+    };
+
+    /* Set up the window manager’s name. According to EWMH, section "Root Window
+     * Properties", to indicate that an EWMH-compliant window manager is
+     * present, a child window has to be created (and kept alive as long as the
+     * window manager is running) which has the _NET_SUPPORTING_WM_CHECK and
+     * _NET_WM_ATOMS. */
+    xcb_window_t child_window = xcb_generate_id(conn);
+    xcb_create_window(
+        conn,
+        XCB_COPY_FROM_PARENT, /* depth */
+        child_window, /* window id */
+        root, /* parent */
+        0, 0, 1, 1, /* dimensions (x, y, w, h) */
+        0, /* border */
+        XCB_WINDOW_CLASS_INPUT_ONLY, /* window class */
+        XCB_COPY_FROM_PARENT, /* visual */
+        0,
+        NULL);
+    xcb_change_property(conn, XCB_PROP_MODE_REPLACE, child_window, A__NET_SUPPORTING_WM_CHECK, XCB_ATOM_WINDOW, 32, 1, &child_window);
+    xcb_change_property(conn, XCB_PROP_MODE_REPLACE, child_window, A__NET_WM_NAME, A_UTF8_STRING, 8, strlen("i3"), "i3");
+    xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root, A__NET_SUPPORTING_WM_CHECK, XCB_ATOM_WINDOW, 32, 1, &child_window);
+
+    /* I’m not entirely sure if we need to keep _NET_WM_NAME on root. */
+    xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root, A__NET_WM_NAME, A_UTF8_STRING, 8, strlen("i3"), "i3");
+
+    xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root, A__NET_SUPPORTED, XCB_ATOM_ATOM, 32, 16, supported_atoms);
 }