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

[3.2] Fix splash screen loading on Android #41384

Closed

Conversation

m4gr3d
Copy link
Contributor

@m4gr3d m4gr3d commented Aug 19, 2020

Fix issue #21824

This is done by leveraging the Android theme to setup the app splash screen (reference).

Note:

  • The splash screen fullsize and use filter parameters are not supported due to limitations of the Android theme api
  • The PR is made against the 3.2-android-app-bundle branch since that branch includes numerous updates to the Android build process.
  • The feature targets the Godot 3.2.4 release.

Aman Jain and others added 14 commits July 9, 2020 16:25
…iles

Utility methods for writing files to gradle project directory
Factor out finding permissions and command line flags
Create strings.xml files for project name translations
Copy project icons to Gradle project directory during Android Custom Build.
Write AndroidManifest.xml for Gradle project
Add 'Export App Bundle' to Android Export Options
@m4gr3d m4gr3d force-pushed the fix_splash_loading_3_2 branch from 992faf4 to 80d8c61 Compare August 19, 2020 22:49
@m4gr3d m4gr3d changed the title Fix splash screen loading on Android [3.2] Fix splash screen loading on Android Aug 19, 2020
@m4gr3d m4gr3d force-pushed the 3.2-android-app-bundle branch 2 times, most recently from 7031c3c to 59da1db Compare September 18, 2020 23:39
@akien-mga akien-mga closed this Sep 27, 2020
@m4gr3d
Copy link
Contributor Author

m4gr3d commented Sep 27, 2020

@akien-mga should I reopen this PR against the 3.2 branch?

@akien-mga
Copy link
Member

Ah, it got closed because the target branch was deleted I guess? Then yes, unless this one can be reopened with its target branch changed.

@akien-mga
Copy link
Member

(Well I see the reopen button is greyed out, so a new PR will be necessary.)

@m4gr3d
Copy link
Contributor Author

m4gr3d commented Sep 28, 2020

Replaced by PR #42389

@akien-mga akien-mga modified the milestones: 3.2, 3.3 Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants