all repos — openbox @ 04e1084d588a7c2aedcaef53d927c33afec17437

openbox fork - make it a bit more like ryudo 

map the appropriate text-containing windows, and unmap the ones that are not being used based on the focus popup cycle mode
Dana Jansens danakj@orodu.net
commit

04e1084d588a7c2aedcaef53d927c33afec17437

parent

9b447a2d2837a6dab0380bc767644f8fe5083aa9

1 files changed, 10 insertions(+), 4 deletions(-)

jump to
M openbox/focus_cycle_popup.c → openbox/focus_cycle_popup.c 
@@ -137,7 +137,6 @@ CWOverrideRedirect | CWBorderPixel, &attrib);
 
     /* create the text window used for the icon-mode popup */
     popup.icon_mode_text = create_window(popup.bg, 0, 0, NULL);
-    XMapWindow(obt_display, popup.icon_mode_text);
 
     popup.targets = NULL;
     popup.n_targets = 0;

@@ -256,7 +255,6 @@ t->iconwin = create_window(p->bg, 0, 0, NULL);
                 t->textwin = create_window(p->bg, 0, 0, NULL);
 
                 XMapWindow(obt_display, t->iconwin);
-                XMapWindow(obt_display, t->textwin);
 
                 p->targets = g_list_prepend(p->targets, t);
                 ++n;

@@ -398,10 +396,14 @@ if (!p->mapped) {
         /* position the background but don't draw it */
         XMoveResizeWindow(obt_display, p->bg, x, y, w, h);
 
-        if (mode == OB_FOCUS_CYCLE_POPUP_MODE_ICONS)
+        if (mode == OB_FOCUS_CYCLE_POPUP_MODE_ICONS) {
             /* position the text */
             XMoveResizeWindow(obt_display, p->icon_mode_text,
                               icon_mode_textx, icon_mode_texty, textw, texth);
+            XMapWindow(obt_display, popup.icon_mode_text);
+        }
+        else
+            XUnmapWindow(obt_display, popup.icon_mode_text);
     }
 
     /* find the focused target */

@@ -451,10 +453,14 @@ XMoveResizeWindow(obt_display, target->iconwin,
                               iconx, icony, HILITE_SIZE, HILITE_SIZE);
 
             /* position the text */
-            if (mode == OB_FOCUS_CYCLE_POPUP_MODE_LIST)
+            if (mode == OB_FOCUS_CYCLE_POPUP_MODE_LIST) {
                 XMoveResizeWindow(obt_display, target->textwin,
                                   list_mode_textx, list_mode_texty,
                                   textw, texth);
+                XMapWindow(obt_display, target->textwin);
+            }
+            else
+                XUnmapWindow(obt_display, target->textwin);
 
             /* get the icon from the client */
             icon = client_icon(target->client, ICON_SIZE, ICON_SIZE);