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

DIRECTOR: Fix visibility issue when setting window properties #5245

Merged
merged 1 commit into from Aug 9, 2023

Conversation

hari01584
Copy link
Contributor

This patch fixes the behaviour of windows when setting their properties (e.g. titleVisibility, Modal, etc) from the lingo.

Initially the window was set to be visible when the property was set, but this is not the expected behaviour. The window should only be visible when the open command is called. Removed setVisible(true) from various window properties assignment functions.

Fixes popupmenu overlay in meet-mediaband and white rectangle when opening mcluhan-win.

--start-movie="main\stvimenu.dxr" mediaband-win

This patch fixes the behaviour of windows when setting their
properties (e.g. titleVisibility, Modal, etc) from the lingo.

Initially the window was set to be visible when the property was
set, but this is not the expected behaviour. The window should
only be visible when the `open` command is called. Removed `setVisible(true)`
from various window properties assignment functions.

Fixes popupmenu overlay in `meet-mediaband` and white rectangle when
opening `mcluhan-win`.

--start-movie="main\stvimenu.dxr" mediaband-win
@sev- sev- added the GSoC Part of a Google Summer of Code project label Aug 8, 2023
@sev-
Copy link
Member

sev- commented Aug 9, 2023

Thank you!

@sev- sev- merged commit 33d4b35 into scummvm:master Aug 9, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GSoC Part of a Google Summer of Code project
Projects
None yet
2 participants