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

Fix stolen mouse input in editor #183

Merged
merged 1 commit into from
Oct 22, 2021

Conversation

HaSa1002
Copy link
Member

@HaSa1002 HaSa1002 commented Oct 16, 2021

Fixes #178
Fixes #180

@HaSa1002 HaSa1002 added the bug Something isn't working label Oct 16, 2021
@HaSa1002 HaSa1002 added this to the 0.9 milestone Oct 16, 2021
@HaSa1002 HaSa1002 added this to In progress in v0.9 (Old) via automation Oct 16, 2021
@HaSa1002 HaSa1002 added the editor everthing that belongs to the editor functions of Libre TrainSim label Oct 17, 2021
@Jean28518
Copy link
Member

quick question: how are mouse clicks handled now in 3D view? Via unhandled input?

@HaSa1002
Copy link
Member Author

So the camera has its own handling in _unhandled_input. Most of the actions are in _unhandled_input as well, except for the dragging. The drag start should be in _unhandled_input. The dragging itself in _input. (I hope I changed it to this logic. Can't remember right now 😅). The selctions will be handled by the physics engine as soon, as this is fixed. Maybe through a signal by the camera, but idk yet tbh.

Copy link
Member

@Jean28518 Jean28518 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@Jean28518 Jean28518 merged commit d38a292 into Libre-TrainSim:master Oct 22, 2021
v0.9 (Old) automation moved this from In progress to Done Oct 22, 2021
@HaSa1002 HaSa1002 deleted the fix-stolen-input branch October 30, 2021 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working editor everthing that belongs to the editor functions of Libre TrainSim
Projects
No open projects
v0.9 (Old)
  
Done
Development

Successfully merging this pull request may close these issues.

EditorHUD is stealing mouse input handle_object_transform errors on deselection
2 participants