Skip to content

feat: android support for ue 5.2 #9

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

Merged
merged 1 commit into from
Sep 14, 2023
Merged

feat: android support for ue 5.2 #9

merged 1 commit into from
Sep 14, 2023

Conversation

nattb8
Copy link
Collaborator

@nattb8 nattb8 commented Sep 13, 2023

Get SDK to run on Android devices. This is for UE 5.2 only.

Changes required:

  • Set up the game bridge after the game engine init loop completes instead of when the viewport is created
  • Instead of waiting for a response for 'init' to come back from the bridge to signal that the browser is ready, use the page load completed event instead. This is because there is some delay in binding all window.ue functions/values (including window.ue.jsconnector).
  • Android webview cannot be collapsed but needs to be at least 1px x 1 px. However, the Android webview is offscreen, so it doesn't overlap with the game.
  • To persist the Android webview across levels, the user widget cannot be removed from the parent
  • Enable DOM storage for Android webview so we can use localStorage

@suluvai
Copy link
Contributor

suluvai commented Sep 14, 2023

Only comment I would add is to add instructions to how to test for Android.

@nattb8
Copy link
Collaborator Author

nattb8 commented Sep 14, 2023

Only comment I would add is to add instructions to how to test for Android.

@suluvai I've added instructions on how to set up Android for Unreal internally.

@nattb8 nattb8 removed the request for review from cajames September 14, 2023 04:25
@nattb8 nattb8 merged commit 2946c81 into main Sep 14, 2023
@nattb8 nattb8 deleted the feat/android-support branch September 14, 2023 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants