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.

Sign up for GitHub

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

Jump to bottom

Remove Move Camera with Timeouts? #47

Closed
mramato opened this issue Feb 4, 2015 · 3 comments
Closed

Remove Move Camera with Timeouts? #47

mramato opened this issue Feb 4, 2015 · 3 comments
Labels
bug

Comments

@mramato
Copy link
Contributor

mramato commented Feb 4, 2015

We should discourage people from using setTimeout in Cesium. Should we remove the Move Camera with Timeouts example? Or we could at least add a comment saying that it's not a recommended way of doing it but preserved because of the port?
@mramato mramato added the bug label Feb 4, 2015
@pjcozzi
Copy link
Contributor

pjcozzi commented Feb 5, 2015

Can we use preRender or another event instead?

@mramato
Copy link
Contributor Author

mramato commented Feb 5, 2015

Maybe, but I'm probably just overthinking it. It's probably better to just keep it a straight thought of the GE example, so I'll close this.

@mramato mramato closed this as completed Feb 5, 2015
@bagnell
Copy link
Contributor

bagnell commented Feb 5, 2015

You could move the camera in the preRender event. It happens right after all of the other automatic camera updates but before rendering. Its one of the main reasons we added it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pjcozzi @bagnell @mramato