Skip to content

Fixes Alt+Enter fullscreen#531

Open
KoopaCode wants to merge 2 commits intosmartcmd:mainfrom
KoopaCode:windows64-fix
Open

Fixes Alt+Enter fullscreen#531
KoopaCode wants to merge 2 commits intosmartcmd:mainfrom
KoopaCode:windows64-fix

Conversation

@KoopaCode
Copy link
Contributor

Description

Fixes fullscreen save so both F11 and Alt+Enter work the same.

Changes

  • Alt+Enter now saves fullscreen state the same way F11 does.
  • Fullscreen now comes back on next launch no matter which key combo was used.

Previous Behavior

Fullscreen only saved correctly when using F11.

If you used Alt+Enter, it went fullscreen for that session, but it did not stay fullscreen next time you launched.

Root Cause

There were two different fullscreen paths.

F11 saved the setting, but Alt+Enter did not save.

New Behavior

Fullscreen now saves and restores correctly for both F11 and Alt+Enter.

Fix Implementation

Made Alt+Enter use the same save path as F11.

  • Both toggle methods now go through the same fullscreen update logic.
  • Fullscreen state is written when either method is used.
  • No other fullscreen behavior was changed.

Related Issues

Fixes ALT+Enter not being persistent like F11
@MCbabel
Copy link
Contributor

MCbabel commented Mar 5, 2026

Yes, that was my mistake, sorry. I completely forgot that you could also do that with Alt+Enter.

@codeHusky
Copy link
Collaborator

Would it make more sense to, instead, observe if the Fullscreen state has changed for any reason, then write it? Seems better than relying strictly on binds.

Observe if the Fullscreen state has changed for any reason and will write it.
@KoopaCode
Copy link
Contributor Author

Would it make more sense to, instead, observe if the Fullscreen state has changed for any reason, then write it? Seems better than relying strictly on binds.

If you check the most recent commit in the PR I have made said changes.

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.

Fullscreen status only persists if fullscreened with F11, not alt+enter

3 participants