don't skip windows that are skip_taskbar unless they are normal typed.. i.e. if a...

diff --git a/openbox/client.c b/openbox/client.c
index 1e2040fd077b1cd13cb17f0b0e615cee64ddc06e..22eb840379787e0de81a0abdd12634d29c7cbe0e 100644 (file)
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -450,6 +450,8 @@ void client_manage(Window window)
     g_free(monitor);
     monitor = NULL;
 
+    ob_debug_type(OB_DEBUG_FOCUS, "Going to try activate new window? %s\n",
+                  activate ? "yes" : "no");
     if (activate) {
         gboolean raise = FALSE;
 

diff --git a/openbox/focus.c b/openbox/focus.c
index df02cb7630bf7c9bff8dbc38f385437ba7f4db37..a4eb2cfa32a76b000c16d6f7d2160519382ed49a 100644 (file)
--- a/openbox/focus.c
+++ b/openbox/focus.c
@@ -337,13 +337,9 @@ gboolean focus_valid_target(ObClient *ft,
                   that can be focused instead */
                !focus_target_has_siblings(ft, iconic_windows, all_desktops))));
 
-    /* it's not set to skip the taskbar (unless it is a type that would be
-       expected to set this hint, or modal) */
-    ok = ok && ((ft->type == OB_CLIENT_TYPE_DOCK ||
-                 ft->type == OB_CLIENT_TYPE_DESKTOP ||
-                 ft->type == OB_CLIENT_TYPE_TOOLBAR ||
-                 ft->type == OB_CLIENT_TYPE_MENU ||
-                 ft->type == OB_CLIENT_TYPE_UTILITY) ||
+    /* it's not set to skip the taskbar (but this only applies to normal typed
+       windows, and is overridden if the window is modal) */
+    ok = ok && (ft->type != OB_CLIENT_TYPE_NORMAL ||
                 ft->modal ||
                 !ft->skip_taskbar);