<p></p>
<blockquote>
<p dir="auto">Another pop up to deal with? Can this be put behind a preference guard (<code>interface.warn_on_replace_in_session</code> in various)?</p>
<pre><code>* Do people _really_ "accidentally" click the button, or is it intentional and they change their minds?  A new popup won't necessarily change the outcome of intentional actions.
</code></pre>
</blockquote>
<p dir="auto">Yes, happened for me twice. The last time it took me an hour or so to clean up my session again (I had to review each single file).</p>
<blockquote>
<pre><code>* The "in session" button is hidden by default, so new users don't see it on first use.
</code></pre>
</blockquote>
<p dir="auto">Geany remembers the fold state of the extra buttons and for me, they are always expanded because I need 'in document' and 'in selection' quite often.</p>
<blockquote>
<pre><code>* Popups like that just train people to click accept/press return twice/etc.
</code></pre>
</blockquote>
<p dir="auto">In some situations yes. Here it is rather a stopper in case you already realised you clicked the wrong button shortly after you clicked it.</p>
<blockquote>
<pre><code>* The change is saved in the undo buffer, so easy to back out.
  
  * Could undo be expanded so that the "in session" action can be undone, rather than the "replace all" in each file?
  * Or add a "reload all" command to reset to the last saved state.
</code></pre>
</blockquote>
<p dir="auto">Both variants only work if there are no othher unsaved changes in session files. Might not always be the case.</p>
<p dir="auto">The main purpose of this confirmation is to make it harder to perform this heavy, very influential operation accidentally.<br>
Other than 'in document' and 'in selection', 'in session' can be hard and time consuming to revert.</p>
<blockquote>
<blockquote>
<p dir="auto">"In Selection" and "In Session" look very similar visually so they are easy to confuse.</p>
</blockquote>
<p dir="auto">Only when people are mindlessly clicking buttons. In that case, a popup isn't going to help.</p>
</blockquote>
<p dir="auto">See above.</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/3033#issuecomment-981125105">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ4RE46LGPX5HZBETYDUOJTY7ANCNFSM5I5OBSHA">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/AAIOWJ4CV3CIU7LFL7YCJSDUOJTY7A5CNFSM5I5OBSHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHJ5MP4I.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/3033#issuecomment-981125105",
"url": "https://github.com/geany/geany/pull/3033#issuecomment-981125105",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>