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

Disable headlock when in VR videos #1737

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Conversation

haanhvu
Copy link
Collaborator

@haanhvu haanhvu commented Jan 30, 2025

VR videos are meant for users to be immersed in the whole space around them. Because headlock locks users in the front of them, it needs to be disabled in VR videos. If users enabled headlock before entering VR videos, we disable it when VR videos show and reenable it when they are exited.

Fixes #1698

Copy link
Member

@svillar svillar left a comment

Choose a reason for hiding this comment

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

Thanks for the patch!

It indeed fixes the issue but the patch can be improved. See comments inline

Copy link
Member

@svillar svillar left a comment

Choose a reason for hiding this comment

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

Almost ready. I think we can make it even better. Comments inline

Copy link
Member

@svillar svillar left a comment

Choose a reason for hiding this comment

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

Almost ready. I think we can make it even better. Comments inline

@haanhvu haanhvu force-pushed the issue1698 branch 2 times, most recently from 02769d6 to ed2d88e Compare February 5, 2025 10:41
VR videos are meant for users to be immersed in the whole space around them. Because headlock locks users in the front of them, it needs to be disabled in VR videos. If users enabled headlock before entering VR videos, we disable it when VR videos show and reenable it when they are exited.

Fixes Igalia#1698
Copy link
Member

@svillar svillar left a comment

Choose a reason for hiding this comment

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

Nice! Thanks for working on this!

@svillar svillar added the release_candidate PR that should be part of the next release label Feb 6, 2025
@svillar svillar merged commit 8a40a50 into Igalia:main Feb 6, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_candidate PR that should be part of the next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable Headlock when starting WebXR/immersive video
2 participants