Don’t set the _NET_WM_WORKAREA hint at all (Thanks cg)

Fixes: #539

diff --git a/include/ewmh.h b/include/ewmh.h
index 0a0cbc292bbdd6bcaf3c87695a7ea461f57b78f1..bde8f319a908bee7140764eaf2c7310ff6db0656 100644 (file)
--- a/include/ewmh.h
+++ b/include/ewmh.h
@@ -28,16 +28,6 @@ void ewmh_update_current_desktop();
  */
 void ewmh_update_active_window(xcb_window_t window);
 
-/**
- * Updates the workarea for each desktop.
- *
- * 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();
-
 /**
  * Updates the _NET_CLIENT_LIST_STACKING hint. Necessary to move tabs in
  * Chromium correctly.

diff --git a/src/con.c b/src/con.c
index 8ecc2401d197a80417c4c0ae2f71f2a03489b72b..f29f9e1d58055c6cefecff9fb5240f360304a3b2 100644 (file)
--- a/src/con.c
+++ b/src/con.c
@@ -1066,7 +1066,6 @@ static void con_on_remove_child(Con *con) {
             LOG("Closing old workspace (%p / %s), it is empty\n", con, con->name);
             tree_close(con, DONT_KILL_WINDOW, false, false);
             ipc_send_event("workspace", I3_IPC_EVENT_WORKSPACE, "{\"change\":\"empty\"}");
-            ewmh_update_workarea();
         }
         return;
     }

diff --git a/src/ewmh.c b/src/ewmh.c
index 85daa433714a190033dee15aa94540c41b6ded79..92ad88b8dc0894d167081f83270f88eb79abf4f0 100644 (file)
--- a/src/ewmh.c
+++ b/src/ewmh.c
@@ -50,6 +50,10 @@ void ewmh_update_active_window(xcb_window_t 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.

diff --git a/src/randr.c b/src/randr.c
index e4bff1ff57e8475d089d4c72d1bcfd4c26a02ad0..d1683e9f925ea7c3c504801be73ff560a9141418 100644 (file)
--- a/src/randr.c
+++ b/src/randr.c
@@ -808,8 +808,6 @@ void randr_query_outputs() {
         disable_randr(conn);
     }
 
-    ewmh_update_workarea();
-
     /* Just go through each active output and assign one workspace */
     TAILQ_FOREACH(output, &outputs, outputs) {
         if (!output->active)

diff --git a/src/workspace.c b/src/workspace.c
index 6bfc7a6dfcc19b9e36463c58e56a52c56c47ddd5..1ccae967d4a76152a71489a7e9eda87b779fb1e4 100644 (file)
--- a/src/workspace.c
+++ b/src/workspace.c
@@ -240,8 +240,6 @@ static void _workspace_show(Con *workspace, bool changed_num_workspaces) {
     }
 
     /* Update the EWMH hints */
-    if (changed_num_workspaces)
-        ewmh_update_workarea();
     ewmh_update_current_desktop();
 
     ipc_send_event("workspace", I3_IPC_EVENT_WORKSPACE, "{\"change\":\"focus\"}");