Skip to content

Disable Game Embedding in Single Window Mode#101936

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
Hilderin:disable-game-embedding-single-window-mode
Jan 24, 2025
Merged

Disable Game Embedding in Single Window Mode#101936
Repiteo merged 1 commit intogodotengine:masterfrom
Hilderin:disable-game-embedding-single-window-mode

Conversation

@Hilderin
Copy link
Copy Markdown
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
Copy Markdown
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
@Repiteo
Copy link
Copy Markdown
Contributor

Repiteo commented Jan 24, 2025

Thanks!

@PiCode9560
Copy link
Copy Markdown
Contributor

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
Copy Markdown
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
Copy Markdown
Contributor

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

@Hilderin
Copy link
Copy Markdown
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