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

Camera jumps when starting to draw a PV system #332

Open
khdlr opened this issue Oct 6, 2024 · 2 comments · May be fixed by #348
Open

Camera jumps when starting to draw a PV system #332

khdlr opened this issue Oct 6, 2024 · 2 comments · May be fixed by #348
Assignees
Labels
bug Something isn't working

Comments

@khdlr
Copy link
Member

khdlr commented Oct 6, 2024

Steps to reproduce:

  1. Simulate building
  2. Rotate/move camera
  3. Click "Neue PV-Anlage einzeichnen"

Expected: Camera view stays the same
Actual: Camera jumps (only sometimes?)

@khdlr khdlr changed the title Camera resets when starting to draw a PV system Camera jumps when starting to draw a PV system Oct 6, 2024
@khdlr khdlr added the bug Something isn't working label Oct 6, 2024
@carwe
Copy link

carwe commented Oct 6, 2024

Reason is, that the center of the building is forced to be at the center of the screen. The user should be trusted that he found camera settings that work for him, the camera should not be changed at this moment.

@FlorianK13
Copy link
Member

As far as I understand this only happens when another building is simulated, and hence SimulationMeshes changes.

@FlorianK13 FlorianK13 self-assigned this Oct 18, 2024
FlorianK13 added a commit that referenced this issue Oct 18, 2024
This makes the code more simple I think
FlorianK13 added a commit that referenced this issue Oct 18, 2024
FlorianK13 added a commit that referenced this issue Oct 18, 2024
FlorianK13 added a commit that referenced this issue Oct 18, 2024
FlorianK13 added a commit that referenced this issue Oct 18, 2024
@FlorianK13 FlorianK13 linked a pull request Oct 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants