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

Window mode fix #51

Merged
merged 2 commits into from
Sep 9, 2024
Merged

Conversation

Heftie
Copy link

@Heftie Heftie commented Oct 20, 2023

The Options were extended to save the last set Width and Height of the window. The UpdateWindow function was modified to use this setting if it is set. When changing the view in options from fullscreen or borderless to windowed it reset to a maximized window, by setting the default window size of 640x480. This will trigger the UpdateWindow function to set it to maximized.
Also, the position was changed to use SDL_WINDOWPOS_CENTERED instead of a calculation.

@WizzardMaker WizzardMaker force-pushed the develop branch 3 times, most recently from 5523803 to ae4ff4e Compare June 27, 2024 20:46
The windowed mode used only the default value of 640 480 every time also on startup. Changed to used maximum window size and centered on screen with SDL constants
The Options were extended to save also the last set Width and Height of the window. The UpdateWindow function was modified to use this setting if it is set. When changing the view in options to windowed its reseted to a maximized window, by setting the default window size of 640x480.
@WizzardMaker
Copy link
Owner

Rebased to the latest HEAD of develop

Thanks for the contribution!

@WizzardMaker WizzardMaker merged commit e760cfd into WizzardMaker:develop Sep 9, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants