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

[Android] Fix the issue causing the logo to not show when using the compatibility renderer #93891

Merged

Conversation

m4gr3d
Copy link
Contributor

@m4gr3d m4gr3d commented Jul 3, 2024

Delaying showing the logo by 1 frame so that it doesn't run in the same frame as the setup logic resolves the issue.

To do so, the logo logic is moved out of Main::setup2(...) into its own Main::setup_boot_logo() method. By default, that method is invoked automatically by Main::setup2(...), but on Android platforms, the default behavior is disabled and the Main::setup_boot_logo() is invoked in the next frame.

Fixes #92966

@m4gr3d m4gr3d added this to the 4.3 milestone Jul 3, 2024
@m4gr3d m4gr3d requested a review from a team as a code owner July 3, 2024 10:16
@m4gr3d m4gr3d requested a review from BastiaanOlij July 3, 2024 10:16
Copy link
Member

@adamscott adamscott left a comment

Choose a reason for hiding this comment

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

The code seems way more comprehensible after this PR. Thanks @m4gr3d !

Otherwise, not to be too much nitpicky, but shouldn't the static const int be in an enum instead?

@m4gr3d m4gr3d force-pushed the fix_logo_not_showing_for_gles3 branch from 9e3e73f to 637f4a1 Compare July 3, 2024 18:17
@m4gr3d m4gr3d requested a review from a team as a code owner July 3, 2024 18:17
@m4gr3d
Copy link
Contributor Author

m4gr3d commented Jul 3, 2024

The code seems way more comprehensible after this PR. Thanks @m4gr3d !

Otherwise, not to be too much nitpicky, but shouldn't the static const int be in an enum instead?

@adamscott I switched it to an enum.

Copy link
Member

@clayjohn clayjohn left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

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

Seems ok.

@akien-mga akien-mga merged commit e6448ca into godotengine:master Jul 4, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks!

@m4gr3d m4gr3d deleted the fix_logo_not_showing_for_gles3 branch July 4, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Very Bad
Development

Successfully merging this pull request may close these issues.

[Android] Splash screen doesn't render with the compatibility renderer
4 participants