<p></p>
<blockquote>
<p><a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1030259376" data-permission-text="Title is private" data-url="https://github.com/geany/geany/issues/2949" data-hovercard-type="pull_request" data-hovercard-url="/geany/geany/pull/2949/hovercard" href="https://github.com/geany/geany/pull/2949">#2949</a> means you can avoid this if you don't want it smile</p>
</blockquote>
<p>Haha... yes...</p>
<blockquote>
<p>IIRC all Geany popups are marked modal, so they should take focus, but it is up to the window manager/wayland compositor to grant that. Seems to work here with Cinnamon.</p>
</blockquote>
<p>It's probably focus stealing prevention...  But I prefer no popups in the first place.</p>
<blockquote>
<p>Ok, but project load does cause a signal that plugins might use, do the project plugins do rescans or other stuff when that happens.</p>
</blockquote>
<p>Since the project is already open, I'd expect that plugins would have done what they needed the first time the project was opened.  I used <code>grep</code> to look for plugins that handle the <code>project-close</code> and <code>project-open</code> signals.</p>
<ul>
<li>
<p>geanylua ­and geanypy – pass the signals on to scripts.</p>
</li>
<li>
<p>projectorganizer – close/open/update project, update sidebar.  I don't see anything that looks like updating indices in response to open/close signals, but those could be hiding in function calls I didn't follow (I only looked at only 2-3 levels across two files).  I've been using this plugin and PR together with no ill effects.</p>
</li>
<li>
<p>scope – Functions that call functions that call functions that look like they eventually update the menu.  Since it's the same project, I would guess the menu state would end up in the same place.</p>
</li>
<li>
<p>geanyctags – Changes widget sensitivity.  Would not be affected by this PR because it's clearly supposed to end up in the same state.</p>
</li>
<li>
<p>treebrowser – Updates the path.  There are also buttons and context menu items that can be used to update the path to achieve similar effect.</p>
</li>
</ul>
<blockquote>
<blockquote>
<p>When a project is opened from the menu, no warning is generated.</p>
</blockquote>
<p>Oh, Ok, well no need to add one then, coulda sworn it did, but maybe its been removed, maybe github search knows ... ahh, it used to for a "recent project" open, but that was removed in <a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/f40569d1c327af162726dbab14a78fd83ca5e0b8/hovercard" href="https://github.com/geany/geany/commit/f40569d1c327af162726dbab14a78fd83ca5e0b8"><tt>f40569d</tt></a>.</p>
</blockquote>
<p>My main concern was opening from the file manager, since that's how I've been opening projects.  Maybe already-open projects can continue to be re-opened from the menu because "Unlikely to be done accidentally." ?</p>
<p>Or the behavior of this PR could be put behind an option, like <code>files.reload_project_open_same</code> ?</p>
<blockquote>
<p>... only getting confused for smarties with multiple paths to the same file, but in that case its no different to the current situation, so no need for real_path.</p>
</blockquote>
<p>I missed the symlink issue.  I've changed it to use the real path.</p>
<blockquote>
<p>Do you get an absolute path from the socket buffer then?</p>
</blockquote>
<p>I think so because the filenames are able to match, preventing the project from opening.</p>
<blockquote>
<p>BTW I am not saying this PR is a bad idea, just want to be sure the implementation has no negative side effects.</p>
</blockquote>
<p>My interpretation is that your intention is to improve the PR...  Or if it's really misguided, help me figure it out...</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/2951#issuecomment-950773232">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ4TGIV2SSOYSDN3MXDUIUWL5ANCNFSM5GS2QFRQ">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&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<img src="https://github.com/notifications/beacon/AAIOWJ5SL67JFCYXAFUTHP3UIUWL5A5CNFSM5GS2QFR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHCV2L4A.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/2951#issuecomment-950773232",
"url": "https://github.com/geany/geany/pull/2951#issuecomment-950773232",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>