[Libreoffice-commits] core.git: vcl/unx

[Armin Le Grand (Allotropia) (via logerrit)]

 vcl/unx/gtk3/gtk3gtkframe.cxx |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit cc1690b9752b3fef3d541de0d26b7c7681615767
Author:     Armin Le Grand (Allotropia) <armin.le.grand at me.com>
AuthorDate: Wed Mar 17 18:31:59 2021 +0100
Commit:     Armin Le Grand <Armin.Le.Grand at me.com>
CommitDate: Fri Mar 19 19:14:43 2021 +0100

    tdf#131031 take MenuBar into account on size change
    
    in GtkSalFrame::SetPosSize when setting local vars for
    maGeometry.nWidth/nHeight the implicitely existing gtk
    native menu bar has to be taken into account. This can
    be done by extracting the values for it and re-apply
    to the new values.
    
    Change-Id: Ic0e76ecfd49fab7dbf330f3e26ab4a6ad3267b35
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112654
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Reviewed-by: Armin Le Grand <Armin.Le.Grand at me.com>

diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx
index bf47097af565..e52ff192869f 100644
--- a/vcl/unx/gtk3/gtk3gtkframe.cxx
+++ b/vcl/unx/gtk3/gtk3gtkframe.cxx
@@ -1553,8 +1553,13 @@ void GtkSalFrame::SetPosSize( tools::Long nX, tools::Long nY, tools::Long nWidth
     {
         m_bDefaultSize = false;
 
+        // tdf#131031 Just setting maGeometry.nWidth/nHeight will delete
+        // the evtl. implicitely existing space at top for the gtk native MenuBar,
+        // will make the Window too big and the StatusBar at the bottom vanish
+        // and thus breaks the fix in tdf#130841.
+        const int nImplicitMenuBarHeight(m_pSalMenu ? m_pSalMenu->GetMenuBarHeight() : 0);
         maGeometry.nWidth = nWidth;
-        maGeometry.nHeight = nHeight;
+        maGeometry.nHeight = nHeight - nImplicitMenuBarHeight;
 
         if( isChild( false ) )
             widget_set_size_request(nWidth, nHeight);