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

Update SDLActivity.java #10854

Merged
merged 1 commit into from
Dec 30, 2024
Merged

Update SDLActivity.java #10854

merged 1 commit into from
Dec 30, 2024

Conversation

quimmedes
Copy link

Fix Full immersive mode in Android 9 and higher.

LAYOUT_IN_DISPLAY_CUTOUT_MODE_SHORT_EDGES
will make sure to use all the screen when the fullscreen flag is enabled.

The other option is
LAYOUT_IN_DISPLAY_CUTOUT_MODE_ALWAYS will work in some cases but in devices that have a camera in the middle this flag will not work.

Description

Existing Issue(s)

Fix Full immersive mode in Android 9 and higher.

LAYOUT_IN_DISPLAY_CUTOUT_MODE_SHORT_EDGES
will make sure to use all the screen when  the fullscreen flag is enabled.

The other option is 
LAYOUT_IN_DISPLAY_CUTOUT_MODE_ALWAYS will work in some cases but in devices that have a camera in the middle this flag will not work.
@sezero sezero requested review from 1bsyl and slouken September 24, 2024 20:42
@1bsyl
Copy link
Contributor

1bsyl commented Sep 25, 2024

I don't really use this flags, so I can't tell what should be used here.

(also, beside the smartphone etc, there are also the VR headsets that should be tested)

@slouken
Copy link
Collaborator

slouken commented Sep 25, 2024

This is on my list to review, thanks!

@slouken slouken added this to the 3.2.0 milestone Sep 25, 2024
@slouken slouken self-assigned this Dec 24, 2024
@slouken slouken merged commit 89977d0 into libsdl-org:SDL2 Dec 30, 2024
40 checks passed
@slouken
Copy link
Collaborator

slouken commented Dec 30, 2024

Merged, thanks!

@slouken slouken modified the milestones: 3.2.0, 2.32.0 Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants