Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SaveState: Various Improvements #11848

Merged
merged 5 commits into from
Oct 1, 2024
Merged

SaveState: Various Improvements #11848

merged 5 commits into from
Oct 1, 2024

Conversation

kamfretoz
Copy link
Contributor

@kamfretoz kamfretoz commented Sep 26, 2024

Description of Changes

This PR adds an option that allows you to configure both the compression type and also the level for savestate.

Preview:
image

Also updates the base FSUI translation and the glyph ranges.

Rationale behind Changes

QoL Improvements and freedom of choices.

Suggested Testing Steps

Test if the savestate are compressed and can be loaded correctly.

@kamfretoz kamfretoz changed the title SaveState: Add configurable type and level SaveState: Add configurable compression type and level Sep 26, 2024
@kamfretoz kamfretoz force-pushed the SaveYourState branch 4 times, most recently from e5e91f8 to 390c33b Compare September 27, 2024 14:40
@kamfretoz kamfretoz marked this pull request as ready for review September 27, 2024 14:42
@kamfretoz
Copy link
Contributor Author

Now ready for review and testing.

Copy link

@AmyRoxwell AmyRoxwell left a comment

Choose a reason for hiding this comment

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

Works on linux! All the compression algorithms work correctly and the compression levels work too.
image

Copy link
Member

@F0bes F0bes left a comment

Choose a reason for hiding this comment

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

Just a couple of formatting issues.

pcsx2/ImGui/FullscreenUI.cpp Outdated Show resolved Hide resolved
pcsx2/ImGui/FullscreenUI.cpp Outdated Show resolved Hide resolved
@kamfretoz
Copy link
Contributor Author

kamfretoz commented Sep 29, 2024

I've added a configurable hotkey to dismiss the savestate selector, as well as shorten the auto-close timeout from 7.5s to just 5s.

I'm open to discussion on what the default hotkey for it will be, for now i've decided to go with Shift+Esc. If anyone has better option, do let me know!

Fobes has recommended a much better and simpler solution, now opening the pause menu will first check if the savestate selector is open and if so, will close it.

The default open timeout is still lowered to 5s.

@kamfretoz kamfretoz changed the title SaveState: Add configurable compression type and level SaveState: Various Improvements Sep 29, 2024
@kamfretoz kamfretoz marked this pull request as ready for review September 29, 2024 15:15
@F0bes F0bes merged commit c1eecb3 into PCSX2:master Oct 1, 2024
12 checks passed
@kamfretoz kamfretoz deleted the SaveYourState branch October 1, 2024 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants