@ntrel commented on this pull request.


> @@ -628,7 +628,9 @@ static GeanyDocument *document_create(const gchar 
> *utf8_filename)
        {
                doc = document_get_current();
                /* remove the empty document first */
-               if (doc != NULL && doc->file_name == NULL && ! doc->changed)
+               if (doc != NULL && doc->file_name == NULL && ! doc->changed &&
+                       // check save as dialog isn't running
+                       gtk_window_is_active(GTK_WINDOW(main_widgets.window)))

So it looks like this function would be a solution (comparing the result to the 
main window):
https://docs.gtk.org/gtk3/method.Application.get_active_window.html

But we don't use GtkApplication, also I'm not sure if it would mean having to 
update all our window creation code too.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3415#discussion_r1118610712
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3415/review/[email protected]>

Reply via email to