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

Wild camera spin with mouse over of the Kerbal Konstructs Base Manager UI in the tracking station #3390

Closed
Growflavor opened this issue Jul 5, 2022 · 1 comment · Fixed by #3392
Milestone

Comments

@Growflavor
Copy link

Growflavor commented Jul 5, 2022

Mouse over of the Kerbal Konstructs Base Manager UI (via Kerbin Side Remastered) in the tracking station results in wild camera spin as shown in this video:

https://youtu.be/bUjX9hdigaE

Please let me know if this is fixed on the Principia side or on the Kerbal Konstructs side.

My recollection is that other various UI windows also once caused the camera to spin wildly with Principia & were somehow fixed but I could only find the issue #2425.

Thanks!

@eggrobin
Copy link
Member

eggrobin commented Jul 5, 2022

This is a long-standing issue which has been reported about windows of multiple mods, and about the debug menu; until yesterday I did not have the faintest idea what was going on.
Yesterday (well, today, I guess) while investigating the possibility scroll wheel interaction as part of #3382 I added the camera controls to Principia’s input locks (so that you don’t zoom the camera while scrolling over the Principia UI) and that led to the same problem.

It turns out that the camera rotation does not get updated when the camera controls are locked, so that we repeatedly apply our change of camera reference frame, leading to wild spin.
This will be fixed in Hilbert.

@eggrobin eggrobin mentioned this issue Jul 6, 2022
@pleroy pleroy added this to the Hilbert milestone Jul 25, 2022
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.

3 participants