[1.1.100] Load dialog constantly scrolls back to selected item while something is writing in the saves folder

[mrvn]

I'm copying a large save game over a slow connection (simulate with rsync --bwlimit) into the saves folder. At the same time I'm trying to load a game in factorio. When I try to scroll through the available games it always jumps right back to have the currently selected game visible making it appear as if scrolling is impossible.

My guess is the logic that detects new games in the saves folder also triggers when the size of a file changes. But why scroll to show the currently selected game when a new game is added to the saves folder at all?

[Rseding91]

Thanks for the report. The reason it scrolls back to the selection is due to how the GUI refreshes the list. When file contents change it refreshes by deleting everything in the list and then re-adding them. After it finishes that it scrolls to the selected save file. So if the files are constantly changing it is constantly refreshing and re-scrolling to the selected save file.

I don't consider it worth the extra GUI code to try to avoid this since changing saves is not normally a slow operation.

[mrvn]

Why can't it remember the item visible at the top instead of scrolling to the selected item?

[Rseding91]

Why can't it remember the item visible at the top instead of scrolling to the selected item?

Because that's not the default, and doing that is exactly the "extra GUI code" I want to avoid. If you want to avoid it, sync with a different folder and move to the actual saves folder when it's done downloading.

[mrvn]

It's creating a hidden file with a random extension and only at the end it gets renamed to the real, non-hidden name. So I'm a bit surprised the game doesn't ignore it.

[Rseding91]

It's creating a hidden file with a random extension and only at the end it gets renamed to the real, non-hidden name. So I'm a bit surprised the game doesn't ignore it.

The game can open saves that have been extracted into folders so the list of file types it needs to care about is more than just .zip. But in general it's just a simple "something in the saves folder changed, refresh the GUI".