-
Notifications
You must be signed in to change notification settings - Fork 25
Save Entries Refresh Info Fix #384
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
Conversation
PringlesGang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you just shouldn't be able to change the CurrentlyFocusedElement as it is saving period. Does an early return in UpdateInput and a guard for SavePages->at(*CurrentPage)->Update(dt) on GetFlag(SF_SAVEICON) or SaveSystem::GetLoadStatus() work as well?
If it doesn't, then this is perfectly fine
85e1c60 to
07c1bbb
Compare
07c1bbb to
e768d4d
Compare
|
Now if you cancel save overwriting it will just freeze up |
539edb8 to
a74e9b5
Compare
- Unfocus sysmesbox only when completely hidden - Remove redundant MainItems field
a74e9b5 to
8563242
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fix fixes the thing that is broken without breaking more things that would require a fix.
PringlesGang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works and lgtm
Uh oh!
There was an error while loading. Please reload this page.