search

andrei-drexler / ironwail

High-performance QuakeSpasm fork
GNU General Public License v2.0
481 stars 43 forks source link

Added quicksave to list of saves in Load and Save menus #238

Closed nolemretaWxd closed 7 months ago

nolemretaWxd commented 12 months ago

I've added the quicksave slot to list of saves

Flecked42 commented 11 months ago

I don't really see the point of doing this.

The purpose of a quicksave is so you can save/load it with a hotkey.

nolemretaWxd commented 11 months ago

I don't really see the point of doing this.

The purpose of a quicksave is so you can save/load it with a hotkey.

It doesn't remove option to load with a hotkey. I can remove the option of saving to that slot

nolemretaWxd commented 11 months ago

The code duplication looks horrible. Everything should be done in the existing for loop, with special case for name formatting when i == 0.

Done, game now also indicates the quicksave slot. obraz Waiting with commiting until wherever to allow manual saving to quicksave slot .

4LT commented 11 months ago

It would be a good idea to create a tracking issue for this, if only to gauge interest in the feature.

What's the behavior of writing a save to the quicksave slot from the save menu? Will that mean quickload will load that save instead of the last quicksave? I'm not sure that's desirable behavior.

nolemretaWxd commented 11 months ago

It would be a good idea to create a tracking issue for this, if only to gauge interest in the feature.

What's the behavior of writing a save to the quicksave slot from the save menu? Will that mean quickload will load that save instead of the last quicksave? I'm not sure that's desirable behavior.

Saving to the quicksave slot from the save menu saves to quick.sav which is the same file that quicksave uses on F6/F9