X-Git-Url: https://git.brokenzipper.com/gitweb?a=blobdiff_plain;f=openbox%2Fevent.c;h=ab0f8d8b3945563b5e2e0dd73c64b182a94b2b0a;hb=2bdccde5c266ad00e7f50c209397f83bcec74e59;hp=62d317f770873ad94f47c25bef38a0ca25c9fc20;hpb=89142ea6b4049d7d7471a1b0aa230be3a4aeef89;p=chaz%2Fopenbox

diff --git a/openbox/event.c b/openbox/event.c
index 62d317f7..ab0f8d8b 100644
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -15,6 +15,9 @@
 #include <X11/Xlib.h>
 #include <X11/keysym.h>
 #include <X11/Xatom.h>
+#ifdef HAVE_SYS_SELECT_H
+#  include <sys/select.h>
+#endif
 
 static void event_process(XEvent *e);
 static void event_handle_root(XEvent *e);
@@ -119,6 +122,9 @@ void event_process(XEvent *e)
 
     /* pick a window */
     switch (e->type) {
+    case MapRequest:
+	window = e->xmap.window;
+	break;
     case UnmapNotify:
 	window = e->xunmap.window;
 	break;
@@ -213,26 +219,37 @@ void event_process(XEvent *e)
 	event_lasttime = e->xproperty.time;
 	break;
     case FocusIn:
+	if (e->xfocus.mode == NotifyGrab) return;
+/* 	if (e->type == FocusIn && window == focus_backup && focus_client != NULL) {
+*/
+            /* Something's focused but we got a focus event for the backup
+               window. this means that something unfocused before we received
+               the new FocusIn. Just ignore it, and refocus what should be
+               focused! */
+/*            client_focus(focus_client);
+            return;
+        }
+*/
+        break;
     case FocusOut:
-	if (e->xfocus.mode == NotifyGrab)
+	if (e->xfocus.mode == NotifyGrab) return;
 	    /*|| e.xfocus.mode == NotifyUngrab ||*/
-	       
 	    /* From Metacity, from WindowMaker, ignore all funky pointer
 	       root events. Its commented out cuz I don't think we need this
 	       at all. If problems arise we can look into it */
 	    /*e.xfocus.detail > NotifyNonlinearVirtual) */
-	    return; /* skip me! */
-	if (e->type == FocusOut) {
-	    /* FocusOut events just make us look for FocusIn events. They
-	       are mostly ignored otherwise. */
-	    XEvent fi;
-	    if (XCheckTypedEvent(ob_display, FocusIn, &fi)) {
+
+        /* FocusOut events just make us look for FocusIn events. They
+           are mostly ignored otherwise. */
+        {
+            XEvent fi;
+            if (XCheckTypedEvent(ob_display, FocusIn, &fi)) {
 		event_process(&fi);
 		/* dont unfocus the window we just focused! */
 		if (fi.xfocus.window == e->xfocus.window)
 		    return;
 	    }
-	}
+        }
 	break;
     case EnterNotify:
     case LeaveNotify:
@@ -247,10 +264,12 @@ void event_process(XEvent *e)
     client = g_hash_table_lookup(client_map, (gpointer)window);
 
     /* deal with it in the kernel */
-    if (client) {
+    if (client)
 	event_handle_client(client, e);
-    } else if (window == ob_root)
+    else if (window == ob_root)
 	event_handle_root(e);
+    else if (e->type == MapRequest)
+	client_manage(window);
     else if (e->type == ConfigureRequest) {
 	/* unhandled configure requests must be used to configure the
 	   window directly */
@@ -283,10 +302,6 @@ static void event_handle_root(XEvent *e)
     Atom msgtype;
      
     switch(e->type) {
-    case MapRequest:
-	g_message("MapRequest on root");
-	client_manage(e->xmap.window);
-	break;
     case ClientMessage:
 	if (e->xclient.format != 32) break;
 
@@ -319,24 +334,22 @@ static void event_handle_client(Client *client, XEvent *e)
      
     switch (e->type) {
     case FocusIn:
-	client->focused = TRUE;
-	engine_frame_adjust_focus(client->frame);
-
-	/* focus state can affect the stacking layer */
-	client_calc_layer(client);
-
         if (focus_client != client)
             focus_set_client(client);
-	break;
-    case FocusOut:
-	client->focused = FALSE;
-	engine_frame_adjust_focus(client->frame);
 
-	/* focus state can affect the stacking layer */
-	client_calc_layer(client);
+        /* focus state can affect the stacking layer */
+        client_calc_layer(client);
 
+        engine_frame_adjust_focus(client->frame);
+	break;
+    case FocusOut:
 	if (focus_client == client)
 	    focus_set_client(NULL);
+
+        /* focus state can affect the stacking layer */
+        client_calc_layer(client);
+
+        engine_frame_adjust_focus(client->frame);
 	break;
     case ConfigureRequest:
 	g_message("ConfigureRequest for window %lx", client->window);
@@ -447,10 +460,16 @@ static void event_handle_client(Client *client, XEvent *e)
 	client_unmanage(client);
 	break;
     case MapRequest:
-	/* we shouldn't be able to get this unless we're iconic */
-	g_assert(client->iconic);
-
-	/*HOOKFIRECLIENT(requestactivate, client);XXX*/
+        if (screen_showing_desktop)
+            screen_show_desktop(FALSE);
+        client_iconify(client, FALSE, TRUE);
+        if (!client->frame->visible)
+            /* if its not visible still, then don't mess with it */
+            break;
+        if (client->shaded)
+            client_shade(client, FALSE);
+        client_focus(client);
+        stacking_raise(client);
 	break;
     case ClientMessage:
 	/* validate cuz we query stuff off the client here */