From: Dana Jansens <danakj@orodu.net>
Date: Wed, 13 Nov 2002 21:21:48 +0000 (+0000)
Subject: fix for windows changing from non-resizable to resizable
X-Git-Tag: openbox-2_1_3^2~16
X-Git-Url: http://git.openbox.org/?a=commitdiff_plain;h=9c82c573b068a331474305575a3fb9cd3a53b420;p=dana%2Fopenbox.git

fix for windows changing from non-resizable to resizable
---

diff --git a/CHANGELOG b/CHANGELOG
index 85e43cf2..d26bd296 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,6 +1,9 @@
 Changelog for Openbox:
 
 2.2.1:
+ * Fix for clients changing from a non-resizable window (Ben Jansens)
+   to a resizable window.
+
  * Fix bug with mutltibyte fonts being really tiny.     (Ben Jansens)
 
  * Stop using Utf8 drawing routines for Xft. These      (Ben Jansens)
diff --git a/src/Window.cc b/src/Window.cc
index 83adf766..25f731e9 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -2957,23 +2957,22 @@ void BlackboxWindow::propertyNotifyEvent(const XPropertyEvent *pe) {
   case XA_WM_NORMAL_HINTS: {
     getWMNormalHints();
 
-    if ((client.normal_hint_flags & PMinSize) &&
-        (client.normal_hint_flags & PMaxSize)) {
-      // the window now can/can't resize itself, so the buttons need to be
-      // regrabbed.
-      ungrabButtons();
-      if (client.max_width <= client.min_width &&
-          client.max_height <= client.min_height) {
-        functions &= ~(Func_Resize | Func_Maximize);
-      } else {
-        if (! isTransient())
-          functions |= Func_Maximize;
-        functions |= Func_Resize;
-      }
-      grabButtons();
-      setAllowedActions();
-      setupDecor();
+    // the window now can/can't resize itself, so the buttons need to be
+    // regrabbed.
+    ungrabButtons();
+    if (((client.normal_hint_flags & PMinSize) &&
+         (client.normal_hint_flags & PMaxSize)) &&
+        (client.max_width <= client.min_width &&
+         client.max_height <= client.min_height)) {
+      functions &= ~(Func_Resize | Func_Maximize);
+    } else {
+      if (! isTransient())
+        functions |= Func_Maximize;
+      functions |= Func_Resize;
     }
+    grabButtons();
+    setAllowedActions();
+    setupDecor();
 
     Rect old_rect = frame.rect;