<p></p>
<blockquote>
<p dir="auto">So from then on its a message meaning "project save failed" that can be generated for many reasons, not just permissions.</p>
</blockquote>
<p dir="auto">I refer to permissions and read-only files because those are the most common conditions that produce popups.  The messages can be produced several times a minute, even seconds apart, depending on how often the user saves files, switches documents, or performs other activities that prompt project saving.  This makes Geany unusable.</p>
<blockquote>
<p dir="auto">And as I said above, saving geany.conf actually doesn't check for failure, so making it do so is a "good thing".</p>
</blockquote>
<p dir="auto">You seemed averse to changing the project popups messages.  I suggested unifying config file behavior as a compromise to add checking for files that aren't checked, while reducing the intrusiveness of the current checks.</p>
<blockquote>
<p dir="auto">I would advise patience (yeah, I'm not good at it either) until the session-split branch is merged to avoid conflicts and re-bases and then you can do all three files at once.</p>
</blockquote>
<p dir="auto">With the session-split branch (last time I looked at it), keys have to be moved from the current save/load functions to the session variants.  To add a preference requires changing code in at least three locations (struct, save, load).  To move a preference to a session requires changing four (current save/load and session save/load).</p>
<p dir="auto">In the long run, it would be better to migrate to the stash system.  The current near-duplicate save/load code would become a single "add".  Then the stash system could be extended with session support.  The add/save/load functions could take a bool or have session variants.  Since the stash system uses pointers to the preference variables, no other changes would be required.</p>
<blockquote>
<p dir="auto">so it would show at least once per session.</p>
</blockquote>
<p dir="auto">Various preferences.</p>
<blockquote>
<p dir="auto">Just because one project fails due to deliberate permissions does not mean the next one opened is the same, so maybe the flag should be reset when projects change.</p>
</blockquote>
<p dir="auto">The changes I've suggested do <em>not</em> eliminate user notification.  It makes them less intrusive.  Users would be able to more easily ignore them.  But that's up to them.</p>
<blockquote>
<p dir="auto">That kind of suggests a separate flag for preferences/session and for projects.</p>
</blockquote>
<p dir="auto">That would be fine with me.</p>
<blockquote>
<p dir="auto">Or alternatively the warning that prefs or project failed to save could be provided by a gtk_info_bar which shows at the top of the edit pane but is not modal and doesn't interrupt the user editing like a dialog does...</p>
</blockquote>
<p dir="auto">That could also work.</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/issues/2993#issuecomment-974719089">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJYCULFRNDQGPPZJVKTUNAMVZANCNFSM5HWOZOCQ">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/AAIOWJ3PVYKBPLZXLV2PFZ3UNAMVZA5CNFSM5HWOZOC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHIMQQ4I.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/issues/2993#issuecomment-974719089",
"url": "https://github.com/geany/geany/issues/2993#issuecomment-974719089",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>