vcl/qt5/QtFrame.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit f7346dc88200fa2160d39b4e4b4ae31fcad59a8c
Author:     Jan-Marek Glogowski <glo...@fbihome.de>
AuthorDate: Sun Jan 9 01:01:49 2022 +0100
Commit:     Jan-Marek Glogowski <glo...@fbihome.de>
CommitDate: Sun Jan 9 12:06:45 2022 +0100

    Qt use platform-independent visibility functions
    
    show() isn't just some conveniently renamed setVisible(true), but
    presents a window in a platform-dependant way, which - at least
    for WASM - means maximized and without a title bar. This is not
    really expected and WASM can also use multiple windows.
    
    Change-Id: I3fbaec1dfc2cd32bf3a90a94dcee01ebc88e58c7
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128179
    Tested-by: Jenkins
    Reviewed-by: Jan-Marek Glogowski <glo...@fbihome.de>

diff --git a/vcl/qt5/QtFrame.cxx b/vcl/qt5/QtFrame.cxx
index 19d9c826f625..46f7a1a409ff 100644
--- a/vcl/qt5/QtFrame.cxx
+++ b/vcl/qt5/QtFrame.cxx
@@ -460,7 +460,7 @@ void QtFrame::Show(bool bVisible, bool bNoActivate)
     if (!bVisible) // hide
     {
         pSalInst->RunInMainThread([this]() {
-            asChild()->hide();
+            asChild()->setVisible(false);
             if (m_pQWidget->isModal())
                 modalReparent(false);
         });
@@ -474,7 +474,7 @@ void QtFrame::Show(bool bVisible, bool bNoActivate)
         QWidget* const pChild = asChild();
         if (m_pQWidget->isModal())
             modalReparent(true);
-        pChild->show();
+        pChild->setVisible(true);
         pChild->raise();
         if (!bNoActivate && !isPopup())
         {

Reply via email to