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

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2586#discussion_r489085713">src/project.c</a>:</p>
<pre style='color:#555'>> +            else
+               {
+                       gtk_widget_grab_focus(e->file_name);
+                       utils_free_pointers(2, project_file_dirname, locale_filename, NULL);
+                       return FALSE;
+               }
</pre>
<blockquote>
<p>I haven't thought about this much (hence the TODO in my example code), are you sure this is the best thing to do?</p>
</blockquote>
<p>Yes, I think it is best this way, because it's the same behaviour as for creating <code>Base path</code>.</p>
<blockquote>
<p>I get that the path is tested above, but what if the directory is created in the meantime (ex. the user leaves the question dialog box up and create the directory using their file manager or the terminal)?</p>
</blockquote>
<p>I just tested the existing logic for <code>Base path</code>, and if the directory is created in the meantime it doesn't recreate the directory. It works. All files inside the directory were preserved.</p>
<blockquote>
<p>I could be wrong, but it seems like falling back to the old behaviour here would be ok, no?</p>
</blockquote>

<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/2586#discussion_r489085713">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ2X2I2AV4CQIF74UCDSF76XLANCNFSM4RLUS2GA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAIOWJ5K6DE6BIJU72ILPXLSF76XLA5CNFSM4RLUS2GKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGODUUGTWI.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/2586#discussion_r489085713",
"url": "https://github.com/geany/geany/pull/2586#discussion_r489085713",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>