M openbox/client.c → openbox/client.c

@@ -607,9 +607,9 @@ frame_client_gravity(self->frame, x, y); /* get where the frame
                                                 would be */
 
     /* XXX watch for xinerama dead areas */
-
-    /* a = screen_area(self->desktop); */
-    a = screen_physical_area_monitor(client_monitor(self));
+    /* This makes sure windows aren't entirely outside of the screen so you
+     * can't see them at all */
+    a = screen_area(self->desktop);
     if (client_normal(self)) {
         if (!self->strut.right && *x >= a->x + a->width - 1)
             *x = a->x + a->width - self->frame->area.width;
@@ -621,9 +621,16 @@ if (!self->strut.top && *y + self->frame->area.height - 1 < a->y)
             *y = a->y;
     }
 
-    if (rude) {
-        /* this is my MOZILLA BITCHSLAP. oh ya it fucking feels good.
-           Java can suck it too. */
+    /* This here doesn't let windows even a pixel outside the screen,
+     * not applied to all windows. Not sure if it's going to stay at all.
+     * I wonder if disabling this will break struts somehow? Let's find out. */
+    if (0 && rude) {
+        /* avoid the xinerama monitor divide while we're at it,
+         * remember to fix the placement stuff to avoid it also and
+         * then remove this XXX */
+        a = screen_physical_area_monitor(client_monitor(self));
+        /* this is ben's MOZILLA BITCHSLAP. "oh ya it fucking feels good.
+           Java can suck it too." */
 
         /* dont let windows map/move into the strut unless they
            are bigger than the available area */