Hi! In 5.2.0 version, but from few versions, I can't link a section from a document to a section from another document anymore. In 5.2.0 version, when I browse to select original document, a select file dialog is displayed but, when I select it, the dialog is still open. :-(
Step by step to reproduce the bug: - Create a file and insert a section (Menu Insert/Section and click to dialog box insert button); - Write some text in this section; - Save your file (Menu File/Save, choose a path and a name and click to save button); - Open a new file and insert a section (Menu Insert/Section), check the link check box and browse to the file you just create before; - A browsing dialog open and you can select your first file. - When you want to click to open button to validate your choice, the browsing dialog still be opened. :-( - But, you can click on "Insert button" into section dialog box and its ok.
Reproducible in LO 5.2 with the GTK3 vclplugin but not with the GTK2 plugin. We have the same problem in Calc when inserting a sheet from a csv file. Seems to be a filepicker related bug. Maxim, does that ring a bell for you? @Ludovic: the command SAL_USE_VCLPLUGIN="gtk" && libreoffice I gave you on irc does not work well with LO 5.2.0.4. To switch to vclplugin GTK2, you need to do export SAL_USE_VCLPLUGIN="gtk" libreoffice in a terminal. Best regards. JBF
Well this seems to do the trick, but no idea how correct it is: diff --git a/vcl/unx/gtk/fpicker/SalGtkPicker.cxx b/vcl/unx/gtk/fpicker/SalGtkPicker.cxx index 968051d..96df4b7 100644 --- a/vcl/unx/gtk/fpicker/SalGtkPicker.cxx +++ b/vcl/unx/gtk/fpicker/SalGtkPicker.cxx @@ -182,10 +182,8 @@ gint RunDialog::run() if (mxToolkit.is()) mxToolkit->removeTopWindowListener(this); -#if !GTK_CHECK_VERSION(3,0,0) if (nStatus != 1) //PLAY gtk_widget_hide( mpDialog ); -#endif Adding Caolán to CC.
I want to not hide right after execute so that if there is a (native gtk) overwrite warning dialog then it has the right parent. But if the dialog is executed and then not destroyed it hangs around of course. Typically its destroyed, but not by this sfx::DocumentInserter thing. Let me try to make DocumentInserter do what I expected to happen, and happens in general.
nah, that seems super difficult to achieve. More sensible seems to hide it at the end of ::execute, but not at the end of each RunDialog::run so it always disappears at the end of ::execute but remains visible and mapped if "do you want to overwrite" is necessary and then goes away after all that is done
Caolán McNamara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=696e83b663d4f3e00f23947613f9f3916a4dd14d Resolves: tdf#101795 restore hiding on end of Gtk fpicker execute It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Hi Caolán, Thank you very much for the fix. I tested it in the master for both cases section linked to another section in Writer and sheet linked to a csv file in Calc, and both work now as expected. Do you plan to backport this fix to 5.2 branch ? Best regards. JBF
yeah, its in the queue as https://gerrit.libreoffice.org/#/c/28642/
Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-5-2": http://cgit.freedesktop.org/libreoffice/core/commit/?id=21188796975be8187e234de446aae4371c3b360a&h=libreoffice-5-2 Resolves: tdf#101795 restore hiding on end of Gtk fpicker execute It will be available in 5.2.2. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.