Skip to content

Add list of save files to save window#822

Merged
notfood merged 4 commits intorwmt:devfrom
ianleeder:new-save-window
Mar 14, 2026
Merged

Add list of save files to save window#822
notfood merged 4 commits intorwmt:devfrom
ianleeder:new-save-window

Conversation

@ianleeder
Copy link
Copy Markdown
Contributor

@ianleeder ianleeder commented Feb 25, 2026

Reasoning

When playing multiple sessions, I forget the name of the existing save file. I figured it might be nice to copy the list of save files like on the Host window, and use that on the Save window.

I copied just about everything from ServerBrowser.cs DrawHost().

Example Host window

image

Existing MP Save window

image

New MP Save window

image image

@ianleeder
Copy link
Copy Markdown
Contributor Author

Hmm, just realised I need to check the button layouts when dev isn't enabled. I suspect the OK button will be off-centre 🤢

@maxsupermanhd
Copy link
Copy Markdown

Yes, we NEED IT! Thanks!

@SaberShip
Copy link
Copy Markdown
Contributor

Well done! I had considered doing this as well but never found the time! Glad to see this nice QOL improvement.

@ianleeder
Copy link
Copy Markdown
Contributor Author

Ok I've adjusted very slightly.

  • Made the window a little narrower, it didn't need to be that wide.
  • Fixed bug where the dev replay button wasn't conditional
  • Centered the Save button if no dev button is shown

Ready for review/merge

image

@ianleeder ianleeder marked this pull request as ready for review February 27, 2026 23:47
Copy link
Copy Markdown
Member

@mibac138 mibac138 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, just a few minor comments

@ianleeder
Copy link
Copy Markdown
Contributor Author

Looks good overall, just a few minor comments

Thanks @mibac138. I've fixed those issues.

@notfood
Copy link
Copy Markdown
Member

notfood commented Mar 2, 2026

Looks fine

@notfood notfood added enhancement New feature or request. 1.6 Fixes or bugs relating to 1.6 (Not Odyssey). labels Mar 2, 2026
@notfood notfood moved this to In review in 1.6 and Odyssey Mar 2, 2026
@notfood notfood merged commit ec4efa4 into rwmt:dev Mar 14, 2026
1 check passed
@github-project-automation github-project-automation bot moved this from In review to Done in 1.6 and Odyssey Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.6 Fixes or bugs relating to 1.6 (Not Odyssey). enhancement New feature or request.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants