]> Dogcows Code - chaz/openbox/blobdiff - openbox/screen.c
change how rc parsing will work. a=b will be parsed in any [section] and given to...
[chaz/openbox] / openbox / screen.c
index 8f60029ab05211db078eedb8aba896da9a1b9e47..3145a63c1d09f269ed8efe31b061da8cbb33cfd5 100644 (file)
@@ -20,7 +20,7 @@
                        SubstructureNotifyMask | SubstructureRedirectMask | \
                        ButtonPressMask | ButtonReleaseMask | ButtonMotionMask)
 
-guint    screen_num_desktops    = 1;
+guint    screen_num_desktops    = 0;
 guint    screen_desktop         = 0;
 Size     screen_physical_size;
 gboolean screen_showing_desktop;
@@ -29,6 +29,7 @@ GPtrArray *screen_desktop_names;
 
 static Rect  *area = NULL;
 static Strut *strut = NULL;
+static Window support_window = None;
 
 static void screen_update_area();
 
@@ -45,7 +46,6 @@ static int another_running(Display *d, XErrorEvent *e)
 gboolean screen_annex()
 {
     XErrorHandler old;
-    Window support;
     pid_t pid;
     int i, num_support;
     Atom *supported;
@@ -68,14 +68,14 @@ gboolean screen_annex()
     PROP_SET32(ob_root, openbox_pid, cardinal, pid);
 
     /* create the netwm support window */
-    support = XCreateSimpleWindow(ob_display, ob_root, 0, 0, 1, 1, 0, 0, 0);
+    support_window = XCreateSimpleWindow(ob_display, ob_root, 0,0,1,1,0,0,0);
 
     /* set supporting window */
-    PROP_SET32(ob_root, net_supporting_wm_check, window, support);
+    PROP_SET32(ob_root, net_supporting_wm_check, window, support_window);
 
     /* set properties on the supporting window */
-    PROP_SETS(support, net_wm_name, utf8, "Openbox");
-    PROP_SET32(support, net_supporting_wm_check, window, support);
+    PROP_SETS(support_window, net_wm_name, utf8, "Openbox");
+    PROP_SET32(support_window, net_supporting_wm_check, window,support_window);
 
     /* set the _NET_SUPPORTED_ATOMS hint */
     num_support = 48;
@@ -153,7 +153,9 @@ void screen_startup()
     /* get the initial size */
     screen_resize();
 
+    screen_num_desktops = 0;
     screen_set_num_desktops(4);
+    screen_desktop = 0;
     screen_set_desktop(0);
 
     /* don't start in showing-desktop mode */
@@ -166,6 +168,15 @@ void screen_startup()
 void screen_shutdown()
 {
     guint i;
+
+    XSelectInput(ob_display, ob_root, NoEventMask);
+
+    PROP_ERASE(ob_root, openbox_pid); /* we're not running here no more! */
+    PROP_ERASE(ob_root, net_supported); /* not without us */
+    PROP_ERASE(ob_root, net_showing_desktop); /* don't keep this mode */
+
+    XDestroyWindow(ob_display, support_window);
+
     for (i = 0; i < screen_desktop_names->len; ++i)
        g_free(g_ptr_array_index(screen_desktop_names, i));
     g_ptr_array_free(screen_desktop_names, TRUE);
@@ -175,10 +186,10 @@ void screen_shutdown()
 
 void screen_resize()
 {
-    /* Set the _NET_DESKTOP_GEOMETRY hint */
     /* XXX RandR support here? */
     int geometry[2];
 
+    /* Set the _NET_DESKTOP_GEOMETRY hint */
     geometry[0] = WidthOfScreen(ScreenOfDisplay(ob_display, ob_screen));
     geometry[1] = HeightOfScreen(ScreenOfDisplay(ob_display, ob_screen));
     PROP_SET32A(ob_root, net_desktop_geometry, cardinal, geometry, 2);
@@ -195,29 +206,11 @@ void screen_resize()
 
 void screen_set_num_desktops(guint num)
 {
-    guint old;
+    guint i, old;
     gulong *viewport;
-     
+    GList *it;
+
     g_assert(num > 0);
-  
-    /* move windows on desktops that will no longer exist! */
-    /* XXX
-       std::list<Client*>::iterator it, end = clients.end();
-       for (it = clients.begin(); it != end; ++it) {
-       unsigned int d = (*it)->desktop();
-       if (d >= num && d != 0xffffffff) {
-       XEvent ce;
-       ce.xclient.type = ClientMessage;
-       ce.xclient.message_type = otk::Property::atoms.net_wm_desktop;
-       ce.xclient.display = **otk::display;
-       ce.xclient.window = (*it)->window();
-       ce.xclient.format = 32;
-       ce.xclient.data.l[0] = num - 1;
-       XSendEvent(**otk::display, _info->rootWindow(), false,
-       SubstructureNotifyMask | SubstructureRedirectMask, &ce);
-       }
-       }
-    */
 
     old = screen_num_desktops;
     screen_num_desktops = num;
@@ -237,6 +230,23 @@ void screen_set_num_desktops(guint num)
     /* may be some unnamed desktops that we need to fill in with names */
     screen_update_desktop_names();
 
+    /* update the focus lists */
+    /* free our lists for the desktops which have disappeared */
+    for (i = num; i < old; ++i)
+        g_list_free(focus_order[i]);
+    /* realloc the array */
+    focus_order = g_renew(GList*, focus_order, num);
+    /* set the new lists to be empty */
+    for (i = old; i < num; ++i)
+        focus_order[i] = NULL;
+
+    /* move windows on desktops that will no longer exist! */
+    for (it = client_list; it != NULL; it = it->next) {
+        Client *c = it->data;
+        if (c->desktop >= num)
+            client_set_desktop(c, num - 1, FALSE);
+    }
+
     dispatch_ob(Event_Ob_NumDesktops, num, old);
 
     /* change our desktop if we're on one that no longer exists! */
@@ -248,23 +258,19 @@ void screen_set_desktop(guint num)
 {
     GList *it;
     guint old;
+    XEvent e;
      
     g_assert(num < screen_num_desktops);
 
-    g_message("Moving to desktop %u", num);
-  
     old = screen_desktop;
     screen_desktop = num;
     PROP_SET32(ob_root, net_current_desktop, cardinal, num);
 
     if (old == num) return;
 
-    /* hide windows from bottom to top */
-    for (it = g_list_last(stacking_list); it != NULL; it = it->prev) {
-        Client *c = it->data;
-       if (c->frame->visible && !client_should_show(c))
-            engine_frame_hide(c->frame);
-    }
+    g_message("Moving to desktop %d", num+1);
+
+    /* show windows before hiding the rest to lessen the enter/leave events */
 
     /* show windows from top to bottom */
     for (it = stacking_list; it != NULL; it = it->next) {
@@ -273,6 +279,19 @@ void screen_set_desktop(guint num)
             engine_frame_show(c->frame);
     }
 
+    /* hide windows from bottom to top */
+    for (it = g_list_last(stacking_list); it != NULL; it = it->prev) {
+        Client *c = it->data;
+       if (c->frame->visible && !client_should_show(c))
+            engine_frame_hide(c->frame);
+    }
+
+    /* focus the last focused window on the desktop, and ignore enter events
+       from the switch so it doesnt mess with the focus */
+    XSync(ob_display, FALSE);
+    while (XCheckTypedEvent(ob_display, EnterNotify, &e));
+    focus_fallback(TRUE);
+
     dispatch_ob(Event_Ob_Desktop, num, old);
 }
 
@@ -356,24 +375,16 @@ void screen_update_desktop_names()
 void screen_show_desktop(gboolean show)
 {
     GList *it;
-    static Window saved_focus = 0;
      
     if (show == screen_showing_desktop) return; /* no change */
 
-    /* save the window focus, and restore it when leaving the show-desktop
-       mode */
-    if (show && focus_client)
-       saved_focus = focus_client->window;
-  
     screen_showing_desktop = show;
 
     if (show) {
        /* bottom to top */
        for (it = g_list_last(stacking_list); it != NULL; it = it->prev) {
            Client *client = it->data;
-           if (client->type == Type_Desktop)
-               client_focus(client);
-           else if (client->frame->visible && !client_should_show(client))
+           if (client->frame->visible && !client_should_show(client))
                 engine_frame_hide(client->frame);
        }
     } else {
@@ -385,15 +396,6 @@ void screen_show_desktop(gboolean show)
        }
     }
 
-    if (!show) {
-       Client *f = focus_client;
-       if (!f || f->type == Type_Desktop) {
-           Client *c = g_hash_table_lookup(client_map,
-                                           (gpointer)saved_focus);
-           if (c) client_focus(c);
-       }
-    }
-
     show = !!show; /* make it boolean */
     PROP_SET32(ob_root, net_showing_desktop, cardinal, show);
 
@@ -402,14 +404,16 @@ void screen_show_desktop(gboolean show)
 
 void screen_install_colormap(Client *client, gboolean install)
 {
+    XWindowAttributes wa;
+
     if (client == NULL) {
        if (install)
            XInstallColormap(ob_display, render_colormap);
        else
            XUninstallColormap(ob_display, render_colormap);
     } else {
-       XWindowAttributes wa;
-       if (XGetWindowAttributes(ob_display, client->window, &wa)) {
+       if (XGetWindowAttributes(ob_display, client->window, &wa) &&
+            wa.colormap != None) {
            if (install)
                XInstallColormap(ob_display, wa.colormap);
            else
@@ -420,14 +424,13 @@ void screen_install_colormap(Client *client, gboolean install)
 
 void screen_update_struts()
 {
-    GSList *it;
+    GList *it;
     guint i;
      
-    if (strut != NULL)
-       g_free(strut);
+    g_free(strut);
     strut = g_new0(Strut, screen_num_desktops + 1);
 
-    for (it = client_list; it; it = it->next) {
+    for (it = client_list; it != NULL; it = it->next) {
        Client *c = it->data;
        if (c->iconic) continue; /* these dont count in the strut */
     
@@ -448,9 +451,8 @@ static void screen_update_area()
 {
     guint i;
     gulong *dims;
-     
-    if (area != NULL)
-       g_free(area);
+
+    g_free(area);
     area = g_new0(Rect, screen_num_desktops + 1);
      
     dims = g_new(unsigned long, 4 * screen_num_desktops);
@@ -494,7 +496,7 @@ static void screen_update_area()
 */
        if (!RECT_EQUAL(old_area, area[i])) {
            /* the area has changed, adjust all the maximized windows */
-           GSList *it;
+           GList *it;
            for (it = client_list; it; it = it->next) {
                Client *c = it->data;
                if (i < screen_num_desktops) {
This page took 0.030927 seconds and 4 git commands to generate.