sc/source/ui/dbgui/PivotLayoutDialog.cxx | 6 ++++++
1 file changed, 6 insertions(+)
New commits:
commit 9f2daed124829717b9350b78ac301a0b9fceacf8
Author: Caolán McNamara <caol...@redhat.com>
AuthorDate: Tue Oct 27 15:59:56 2020 +0000
Commit: Caolán McNamara <caol...@redhat.com>
CommitDate: Tue Oct 27 20:46:18 2020 +0100
tdf#137726 hide pivot table dlg so its not a candidate as error dlg parent
because this pivot table dialog is going to disappear on response and the
error
dialog is not run in its own event loop but instead async so will end up as
a
parent of a dialog that doesn't exist
Change-Id: I78302428a246612e3107d2907ffffb97a2d25ea2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104895
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caol...@redhat.com>
diff --git a/sc/source/ui/dbgui/PivotLayoutDialog.cxx
b/sc/source/ui/dbgui/PivotLayoutDialog.cxx
index 8c92e8378172..2263370a2eec 100644
--- a/sc/source/ui/dbgui/PivotLayoutDialog.cxx
+++ b/sc/source/ui/dbgui/PivotLayoutDialog.cxx
@@ -628,6 +628,12 @@ void ScPivotLayoutDialog::Close()
IMPL_LINK_NOARG( ScPivotLayoutDialog, OKClicked, weld::Button&, void )
{
+ /* tdf#137726 hide so its not a candidate to be parent of any error
+ messages that may appear because this dialog is going to disappear on
+ response(RET_OK) and the error dialog is not run in its own event loop
+ but instead async */
+ m_xDialog->hide();
+
ApplyChanges();
m_xDialog->response(RET_OK);
}
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
