sc/source/ui/dbgui/pvfundlg.cxx | 2 +- sc/uiconfig/scalc/ui/datafieldoptionsdialog.ui | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-)
New commits: commit 75cbd4955628a76511513a1f78dd63b3bf415245 Author: Julien Nabet <serval2...@yahoo.fr> AuthorDate: Tue Jan 12 20:09:40 2021 +0100 Commit: Andras Timar <andras.ti...@collabora.com> CommitDate: Wed Apr 7 23:23:48 2021 +0200 Revert "pivot table: make options cancel button working" + fix crash again In the dialog, you have 3 buttons "Help", "Cancel" and "OK" Using "Cancel" will call "close" which doesn't revert back changes. To test this: - Launch Calc - Create an array with 2 cols (qtX and qtY) and put 2 rows of values - Create a pivot dialog - Put qtX in "Filters" part - Double click on qtX (=> "Data Field" dialog appears) - Click "Options..." (=> "Data Field Options" dialog appears) - Enable any checkbox (eg: Empty line after each time) + click "OK" (=> dialog "Data Field Options" closes) - Click "Options..." again (the checkbox is still enabled as expected) - Disable the checkbox + click "Cancel" (=> dialog "Data Field Options" closes) - Click "Options..." again (the checkbox is still enabled as NOT expected) So revert the below quoted commit and fix this by using "cancel" in: - action-widget response - id for the GtkButton - the use of the id to initialize m_xBtnCancel with m_xBuilder->weld_button This reverts commit c2a349bc1482482d3e5ec06232c166d77fe9b602. Change-Id: I555a90706de635e90b0ee397dfd958df55b5c40a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109193 Tested-by: Jenkins Reviewed-by: Szymon Kłos <szymon.k...@collabora.com> diff --git a/sc/source/ui/dbgui/pvfundlg.cxx b/sc/source/ui/dbgui/pvfundlg.cxx index 2d165c04ee68..45c831e998cd 100644 --- a/sc/source/ui/dbgui/pvfundlg.cxx +++ b/sc/source/ui/dbgui/pvfundlg.cxx @@ -654,7 +654,7 @@ ScDPSubtotalOptDlg::ScDPSubtotalOptDlg(weld::Window* pParent, ScDPObject& rDPObj , m_xFtHierarchy(m_xBuilder->weld_label("hierarchyft")) , m_xLbHierarchy(m_xBuilder->weld_combo_box("hierarchy")) , m_xBtnOk(m_xBuilder->weld_button("ok")) - , m_xBtnCancel(m_xBuilder->weld_button("close")) + , m_xBtnCancel(m_xBuilder->weld_button("cancel")) , mrDPObj(rDPObj) , maLabelData(rLabelData) { diff --git a/sc/uiconfig/scalc/ui/datafieldoptionsdialog.ui b/sc/uiconfig/scalc/ui/datafieldoptionsdialog.ui index c51c0406d53e..f94a3042bec5 100644 --- a/sc/uiconfig/scalc/ui/datafieldoptionsdialog.ui +++ b/sc/uiconfig/scalc/ui/datafieldoptionsdialog.ui @@ -58,7 +58,7 @@ </packing> </child> <child> - <object class="GtkButton" id="close"> + <object class="GtkButton" id="cancel"> <property name="label">gtk-cancel</property> <property name="visible">True</property> <property name="can_focus">True</property> @@ -651,7 +651,7 @@ </child> <action-widgets> <action-widget response="-5">ok</action-widget> - <action-widget response="-6">close</action-widget> + <action-widget response="-6">cancel</action-widget> <action-widget response="-11">help</action-widget> </action-widgets> <child type="titlebar"> _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits