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

KMSDRM: Odd screen handling when toggling between GUI/emulation windows #1411

Closed
giantclambake opened this issue Aug 31, 2024 · 3 comments
Closed
Assignees
Labels
bug fixed in v5 Fixed in v5.x fixed in v6 Fixed in v6.x

Comments

@giantclambake
Copy link

RPi4/bookworm/KMSDRM ~ evident in Master & Preview

To recreate;

At the console issue the cmd ./amiberry AmigaTestKit.adf

-emulation boots showing the AmigaTestKit GUI fullscreen --> hit F12 to pop GUI, hit F12 again to return to emulation

-- AmigaTestKit GUI reappears, no longer fullscreen with H/V uncentered (looks to be in a different resolution)

--> hit F12 to pop GUI -> Display -> uncheck Correct Aspect Ratio --> hit F12 to resume

-- AmigaTestKit GUI reappears, now fullscreen again -> F12, hit F12 again to return to emulation, AmigaTestKit GUI reappears, no longer fullscreen with H/V uncentered

--> hit F12 to pop GUI -> Display -> check Correct Aspect Ratio --> hit F12 to resume

-- AmigaTestKit GUI reappears, now fullscreen again -> F12, hit F12 again to return to emulation, AmigaTestKit GUI reappears, no longer fullscreen with H/V uncentered

...wash, rinse, repeat ;)

TIA

@midwan midwan self-assigned this Aug 31, 2024
@midwan
Copy link
Collaborator

midwan commented Aug 31, 2024

I noticed that after the fix yesterday, but didn't have time to look into it further. It's a side-effect of using a single window, as I don't modify the window if it's already there (so it will use whatever last settings it had).

@giantclambake
Copy link
Author

Ahh, ok...that explains it ~ it's not actually Correct Aspect Ratio perse, it's more the case that enabling/disabling that option, prompts the code to re-evaluate the desired window settings...or such'n'similar.

@midwan midwan added the bug label Aug 31, 2024
midwan added a commit that referenced this issue Aug 31, 2024
Disable integer scaling when opening the GUI, trigger an update of scaling options when resuming
@midwan midwan closed this as completed in 31749aa Aug 31, 2024
@midwan midwan added fixed in v6 Fixed in v6.x fixed in v5 Fixed in v5.x labels Aug 31, 2024
@midwan
Copy link
Collaborator

midwan commented Aug 31, 2024

Should be fixed now, at least according to my tests :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fixed in v5 Fixed in v5.x fixed in v6 Fixed in v6.x
Projects
None yet
Development

No branches or pull requests

2 participants