<p></p>
<p><b>@eht16</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2859#discussion_r685552018">src/callbacks.c</a>:</p>
<pre style='color:#555'>> @@ -335,6 +335,21 @@ void on_toolbutton_reload_clicked(GtkAction *action, gpointer user_data)
        document_reload_prompt(doc, NULL);
 }
 
+/* reload all files */
+void on_toolbutton_reload_all_clicked(GtkAction *action, gpointer user_data)
+{
+       guint i;
+       gint cur_page = gtk_notebook_get_current_page(GTK_NOTEBOOK(main_widgets.notebook));
+       
+       foreach_document(i)
+       {
+               if (! (documents[i]->file_name == NULL))
</pre>
<p>What if we use a single confirmation dialog for the "reload all" feature which just explains the user that Geany is going to reload all files and that any unsaved changes will be lost. The dialog could look like "Reloading all open documents cause any unsaved changes to be lost" [OK] [Cancel] or something like this.<br>
So the user has the chance to cancel the progress before anything is reloaded and only experiences the "annoyance" of one dialog.</p>
<p>The dialog is necessary only if:</p>
<ul>
<li>any of the open documents has unsaved changes</li>
<li><code>keep_edit_history_on_reload</code> is off</li>
</ul>
<p>I think this could work well enough for most users, assuming the majority has <code>keep_edit_history_on_reload</code> on anyway.</p>
<p>(Sorry if I repeated this idea in case one of you mentioned it already earlier.)</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2859#discussion_r685552018">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ7NNTYOF5FBSTJWRLTT4BGGTANCNFSM5BN3PO7A">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email">Android</a>.<img src="https://github.com/notifications/beacon/AAIOWJYBVZEPFLGJEDVH4SDT4BGGTA5CNFSM5BN3PO7KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOFNBZFCY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2859#discussion_r685552018",
"url": "https://github.com/geany/geany/pull/2859#discussion_r685552018",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>