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

viewer.zoomTo rectangle failed #10517

Closed
zouyaoji opened this issue Jul 6, 2022 · 8 comments · Fixed by #10607
Closed

viewer.zoomTo rectangle failed #10517

zouyaoji opened this issue Jul 6, 2022 · 8 comments · Fixed by #10607

Comments

@zouyaoji
Copy link
Author

@ggetz
Is there any follow-up?

@ggetz
Copy link
Contributor

ggetz commented Jul 12, 2022

@zouyaoji I'm planning to take a look this week and will let you know what I find.

@ggetz
Copy link
Contributor

ggetz commented Jul 28, 2022

Hi @zouyaoji, this is actually a duplicate of #5164. I'll close this so that conversation can continue in that original issue. This happens since the callback property makes the geometry dynamic instead of static.

@zouyaoji
Copy link
Author

@ggetz

But in some 1.8x versions, there is no such bug.

@zouyaoji
Copy link
Author

There is no such problem in version 1.89

@zouyaoji
Copy link
Author

@ggetz
Please see this example, in version 1.89, the behavior is normal.

https://codepen.io/zouyaoji/pen/abYVmqE?editors=1111

@ggetz
Copy link
Contributor

ggetz commented Jul 28, 2022

Thanks for your explanation @zouyaoji. I was able to identify the issue and I believe I have a fix in #10607.

@ggetz ggetz moved this from Needs triage to In Review in CesiumJS Issue/PR backlog Jul 28, 2022
@zouyaoji
Copy link
Author

@ggetz
thank you very much for your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Issue/PR closed
Development

Successfully merging a pull request may close this issue.

2 participants