@codebrainz 
I agree that the current implementation is quite arbitrary about when to save. 
Though, in daily work, I guess this fits for most users. And yes, a more 
general config system (as you mentioned) with notices and a queue would be 
awesome but is a whole another story.

@elextr this PR doesn't address the issue with multiple instances which 
overwrite the config mutually. I think we should stop writing the config at all 
in new instances but again, another story.

@elextr I'm not sure how useful a setting for this feature would be. the config 
we write is only a few kilobytes and I have no clue if there are still any 
users out there who actually store configs on network shares or the like.

Oh, and I think this should *not* go into the SaveActions plugin as the plugin 
is for (auto) saving documents not Geany's settings. And actually, the code of 
this PR is triggered by the SaveActions plugin already via the "document-save" 
signal. So what you request is already done :).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2114#issuecomment-477957247

Reply via email to