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

Add hack to deal with videos in Oculus Browser when exiting VR mode #1116

Merged
merged 2 commits into from
Mar 28, 2019

Conversation

gfodor
Copy link
Contributor

@gfodor gfodor commented Mar 27, 2019

This PR adds a hack because Oculus Browser pauses all videos in the scene when exiting VR mode. It also flips on a 1s delay on spawning on for mobile VR browsers because when you exit VR mode the head orientation is locked (so if you were looking up at the hud, the object you spawn will be above your head).

Copy link
Contributor

@johnshaughnessy johnshaughnessy left a comment

Choose a reason for hiding this comment

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

Oculus browser video hack is fine. I don't understand the spawn delay

@@ -458,10 +458,11 @@ export default class SceneEntryManager {
if (entry.type === "scene_listing" && this.hubChannel.permissions.update_hub) return;

// If user has HMD lifted up, delay spawning for now. eventually show a modal
const delaySpawn = isIn2DInterstitial() && !isMobileVR;
const spawnDelay = isIn2DInterstitial() ? (isMobileVR ? 1000 : 3000) : 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

If I recall correctly the problem was that things would spawn up in the air above the user if their headset was on their forehead. If that's the case, fixing the spawn location seems like a better solution than adding a delay

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.

2 participants