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 Recenter functionality #27

Closed
cai-mobfish opened this issue Apr 29, 2020 · 5 comments · Fixed by #41
Closed

Add Recenter functionality #27

cai-mobfish opened this issue Apr 29, 2020 · 5 comments · Fixed by #41
Assignees
Milestone

Comments

@cai-mobfish
Copy link
Contributor

Detail

This is a standard Recenter function, it can be used to make camera face "Front" in scene.

@cai-mobfish cai-mobfish added this to the Beta 0.6 milestone May 8, 2020
@rohantipare
Copy link
Collaborator

@cai-mobfish
I have implemented the Recenter functionality and pushed on branch 'recenterCamera'
Function: RecenterCamera() in CardboardHeadTracker.cs
Scene: 'demoScene - test recenter'
The demo script recenters the camera every 5 seconds.

@cai-mobfish
Copy link
Contributor Author

@rohantipare
Thanks for the implementation, this recenter works, but we also need another kind of recenter, that the camera only "recenter horizontally", can you do an extra method for this?

Also, I did a little change in this branch, the DemoScene also contains a button to recenter now. You can pull this branch before making another changes.

@rohantipare
Copy link
Collaborator

Sure, I will implement the recenter horizontal only functionality

@rohantipare
Copy link
Collaborator

@cai-mobfish I have implemented the Horizontal only recenter. Changes are pushed to branch 'recenterCamera'

@cai-mobfish
Copy link
Contributor Author

cai-mobfish commented May 19, 2020

Tested, works.
Did an extra commit to remove unused files.

Merged into public-dev.

@cai-mobfish cai-mobfish linked a pull request May 19, 2020 that will close this issue
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 a pull request may close this issue.

2 participants