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

New commits:
commit a81c6d0634fbe0a4aa5836f277e4a8065d9a9b19
Author:     Jan-Marek Glogowski <[email protected]>
AuthorDate: Sun Jan 9 01:01:49 2022 +0100
Commit:     Jan-Marek Glogowski <[email protected]>
CommitDate: Wed Apr 6 13:16:58 2022 +0200

    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 <[email protected]>
    (cherry picked from commit f7346dc88200fa2160d39b4e4b4ae31fcad59a8c)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/132624
    Reviewed-by: Michael Weghorn <[email protected]>

diff --git a/vcl/qt5/QtFrame.cxx b/vcl/qt5/QtFrame.cxx
index 76e89ecdb76b..10b7e46909c5 100644
--- a/vcl/qt5/QtFrame.cxx
+++ b/vcl/qt5/QtFrame.cxx
@@ -461,7 +461,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);
         });
@@ -475,7 +475,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