#2192 Load and Save Snapshot File Dialog Default Directory

[Ziggy]

When the load and save snapshot file dialog is opened, it should show the contents of the configured history/snapshot directory instead of the current working directory.

Windows example:
1) Make directory "VICE_Snapshots" in current working dir
2) Run "<vice install="" path="">\bin\x64sc -eventsnapshotdir VICE_Snapshots"
3) Select Preferences -> Settings pulldown menu
4) Select Host -> Snapshot/event/media recording
5) Confirm "History/snapshot directory" is set to "VICE_Snapshots\"
6) Press Alt + L to load a snapshot or Alt + S to save a snapshot
7) Load and save snapshot file dialog shows contents of current working dir instead of "VICE_Snapshots"</vice>

Similarly, I would think we would want quick snapshot files to be loaded/saved using this configured directory as well.

[Ziggy]

Step 2 above should read:
2) Run "x64sc -eventsnapshotdir VICE_Snapshots"

Step 7 above should read:
7) Load and save snapshot file dialog shows contents of current working dir instead of "VICE_Snapshots"

[Ziggy]

Also want to add that I'm using VICE 3.10.

[compyx]

I've fixed the load/save dialogs to respect the event/snapshot directory set in the settings.

It looks like the quickload/quicksave options use the user's config dir (~/.config/vice/ on Unix, probably user profile dir on Windows), which kinda makes sense for "throwaway" files.
But this behavior can be changed to respect the "EventSnapshotDir" resource as well.

[Ziggy]

Wow, thanks for the quick response compyx!

I've done several tests and it now works great with respect to the load/save dialogs. Great work!

For quickload/quicksave, I think I agree with you. They are "throwaway" files which probably shouldn't be mixed with the snapshots saved via save dialog. quicksave files are definitely loaded/saved in the user profile dir on Windows...I tested that as well just now.

I'm very happy with your fix! I think we can close this ticket now if you agree!