jpage8580 / GTUltra

GTUltra - Extensively modified GoatTracker Stereo (2.76) version. With many new features. See the GTUltra.PDF file for full details
GNU General Public License v2.0
88 stars 10 forks source link

[Request] Add a "File already exists. Replace?" dialog when saving over an existing .sng #25

Closed acrouzet closed 2 years ago

acrouzet commented 2 years ago

Me and others have saved over projects with blanks when trying to load numerous times.

jpage8580 commented 2 years ago

Is this due to accidentally pressing F11 instead of F10? I changed the panel colour to try to reduce this happening (making it clearer that you were in the save panel).

But, yes, I could add a “are you sure?” If you try to save over a different file than the one you last loaded.

On Wed, 25 May 2022 at 20:37, acrouzet @.***> wrote:

Me and others have saved over projects with blanks when trying to load numerous times.

— Reply to this email directly, view it on GitHub https://github.com/jpage8580/GTUltra/issues/25, or unsubscribe https://github.com/notifications/unsubscribe-auth/AYGMDBRL72UGEYZBHYTGJILVLZ6QJANCNFSM5W6G3P2A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

acrouzet commented 2 years ago

Is this due to accidentally pressing F11 instead of F10? I changed the panel colour to try to reduce this happening (making it clearer that you were in the save panel).

Yes, it is, but sometimes the color difference just isn't clear enough, plus not working for colorblind ppl

jpage8580 commented 2 years ago

Hats a very good point! My father is actually red/green colour blind. I’ll add another check,as said before.

I was also thinking of auto-saving to a temp file every, say, 30 seconds. (Maybe keeping the last 10 saves). Could be useful if there’s any accidents or crashes..

On Thu, 26 May 2022 at 21:26, acrouzet @.***> wrote:

Is this due to accidentally pressing F11 instead of F10? I changed the panel colour to try to reduce this happening (making it clearer that you were in the save panel).

Yes, it is, but sometimes the color difference just isn't clear enough, plus not working for colorblind ppl

— Reply to this email directly, view it on GitHub https://github.com/jpage8580/GTUltra/issues/25#issuecomment-1138995758, or unsubscribe https://github.com/notifications/unsubscribe-auth/AYGMDBVFTNZUV6IKMFZVD5DVL7M77ANCNFSM5W6G3P2A . You are receiving this because you commented.Message ID: @.***>

jpage8580 commented 2 years ago

Done. Let me know if that's OK for you

acrouzet commented 2 years ago

That looks alright, thank you!