fixed bug #1717782, Bad fullscreen in ooimpress.

2 files changed, 16 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index db79872..9f6f4b6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
  (Format: Year/Month/Day)
 Changes for 1.1:
+*07/05/20:
+   * Fixed bug #1717782, Bad fullscreen in ooimpress2  ( Henrik )
+     mwm hint had no-resize which made resizing with 
+     FluxboxWindow::setFullscreen impossible.
+     Window.cc
 *07/05/19:
    * Changed behavior of resource session.screen<N>.followModel (Mark)
      - now only options are Ignore and Follow, the latter using the setting in
diff --git a/src/Window.cc b/src/Window.cc
index 68aef8d..3042812 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -1605,9 +1605,18 @@ void FluxboxWindow::setFullscreen(bool flag) {
         // clear decorations
         setDecorationMask(0);
 
+        // dont call Window::moveResize here, it might ignore the 
+        // resize if win state is not resizable; 
+        // instead we call frame resize directly
+        // (see tests/fullscreentest.cc)
+
         // be xinerama aware
-        moveResize(screen().getHeadX(head), screen().getHeadY(head),
-                   screen().getHeadWidth(head), screen().getHeadHeight(head));
+        frame().moveResize(screen().getHeadX(head), screen().getHeadY(head),
+                           screen().getHeadWidth(head), screen().getHeadHeight(head));
+        sendConfigureNotify();
+        m_last_resize_x = frame().x();
+        m_last_resize_y = frame().y();
+
         moveToLayer(::Layer::ABOVE_DOCK);
 
         fullscreen = true;