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

WebXR Support #49

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

andresgutierrez17
Copy link

I created this pull request to add WebXR support to Ultimate XR. I tried to maintain the same programming structure without affecting anything. I had to modify the Avatar prefab to include the necessary scripts for WebXR and add a script that uses the Microphone class, which doesn't exist in the WebGL platform. The idea is to install the WebXR package from de-panther, which can be found at https://openupm.com/packages/com.de-panther.webxr/.

To make it work, you'll need to install that package, import the WebXR templates it provides, and export for WebGL.

In this version I added WebXR SDK compatibility.
Added auto-create WebXRManager and WebXRControllers.
I added comments to the code to explain what it does.
I removed unused script that I created
Updated UxrWebXRControllerInput.cs, fixed issue if WebXR is not installed
@entromp
Copy link
Collaborator

entromp commented Jul 26, 2023

Hi @andresgutierrez17,

Thanks for such an amazing contribution, having WebXR support is insane!
Let me review the code these days and hopefully we can merge it to the main branch.

Thanks again!

@entromp
Copy link
Collaborator

entromp commented Nov 30, 2023

We're currently finishing version 1.0 and will switch to GitHub for internal development too. We expect to merge this into 1.0 during December. Hang in there! :)

@andresgutierrez17
Copy link
Author

Wow, I saw what you were doing for the new release and I am excited about this. I want to use the new version of UltimateXR soon. Will be insane that you'll use my pull request.

@sollaholla
Copy link

There's a portion of this commit that's fairly important for my project, specifically the fact that you can't build for WebGL since the "Microphone" class is not available when the UNITY_WEBGL symbol is active.

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