-
-
Notifications
You must be signed in to change notification settings - Fork 97
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
Use the the system's DeviceType for the generic input profile #1019
Conversation
if (mActiveMapping->controllerType == device::UnknownType) | ||
delegate.SetControllerType(mIndex, DeviceUtils::GetDeviceTypeFromSystem(false)); | ||
else | ||
delegate.SetControllerType(mIndex, mActiveMapping->controllerType); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I prefer this to happen in OpenXRInputSource::Initialize()
. We can directly modify the mapping that is added to the mMappings
vector.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, done.
490ed63
to
800b415
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there
The generic input profile has the device::UknomnType enumeraton. This PR changes it by the one provided by Android.
800b415
to
b4dfb9f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
The generic input profile is associated to the device::UnknownType which uses an empty VR controller in mozilla. The consequence is that there no controller is drawn during WebXR experiences for devices using the generic profile (eg Lynx).
This issue had been fixed already in PR #834, but it got reverted because it broke the logic to detect generic profiles as fallbacks (see PR #907)
The solution proposed in this PR is to assign the DeviceType provided by the system to the ControllerDelegate instance, instead of the one provided by the generic profile.
Fixes #833