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

wl: Fix the COG_PLATFORM_WL_VIEW_FULLSCREEN not works #676

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

psaavedra
Copy link
Member

The COG_PLATFORM_WL_VIEW_FULLSCREEN is evaluated during inside of the cog_wl_viewport_create_window(). At that point there is not yet a view created in the viewport associated to the window.

Allow the shells to set the fullscreen mode even if there is not a view available in the viewport. This will make the new views being created in fullscreen mode. This applies also to the one created during the initialization.

The COG_PLATFORM_WL_VIEW_FULLSCREEN is evaluated during inside of the
cog_wl_viewport_create_window(). At that point there is not yet a view
created in the viewport associated to the window.

Allow the shells to set the fullscreen mode even if there is not a view
available in the viewport. This will make the new views being created
in fullscreen mode. This applies also to the one created during the
initialization.
@psaavedra psaavedra self-assigned this Nov 24, 2023
@psaavedra psaavedra added the bug Something isn't working label Nov 24, 2023
@psaavedra psaavedra requested a review from zhani November 24, 2023 09:29
@psaavedra psaavedra merged commit 6a58209 into master Nov 24, 2023
5 checks passed
@psaavedra psaavedra deleted the psaavedra/fix_COG_PLATFORM_WL_VIEW_FULLSCREEN branch November 24, 2023 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants