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

Disable Game Embedding in Single Window Mode #101936

Merged

Conversation

Hilderin
Copy link
Contributor

@Hilderin Hilderin commented Jan 22, 2025

This PR disables Game Embedding in Single Window Mode. It is currently not functioning well due to the inability to display popups or tooltips in front of the game window. On top of that, embedding mode is now the default.

While this is not a true fix for these issues, I do not have a better solution at the moment. At the very least, users will no longer need to disable embedding to run their games properly. A more suitable solution is required to correctly embed the game window in Single Window Mode.

Copy link
Member

@bruvzg bruvzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not think this can fully be fixed in single-window mode, so it seems reasonable to disable it.

@Repiteo Repiteo merged commit b298f6f into godotengine:master Jan 24, 2025
19 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Jan 24, 2025

Thanks!

@PiCode9560
Copy link

PiCode9560 commented Jan 27, 2025

Can't you just display the game screen in the editor instead of on a separate window?

I mean like display the game screen on subviewport in the editor.

@Hilderin
Copy link
Contributor Author

The separate window qas already disabled in Single window mode. The issues were when the game was inside the editor. Using a subviewport is another project already known to make it work in Single window mode and on Mac.

@PiCode9560
Copy link

So you're going to make single window mode use subviewport in the future?

@Hilderin
Copy link
Contributor Author

Not me personnally because I don't have the knownlegde, but yes, it's the objective.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants