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

Mouse input Events & deselect on void click #62

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

emyxiv
Copy link
Contributor

@emyxiv emyxiv commented Dec 5, 2022

Deselect Gizmo when click on nothing.

Known issue: the mouse click events aren't prevented when clicking on "Group Pose Settings" vanilla window.

@emyxiv emyxiv force-pushed the mouse-input branch 2 times, most recently from 35fbbf5 to 0e550b9 Compare December 6, 2022 23:03
@emyxiv
Copy link
Contributor Author

emyxiv commented Dec 6, 2022

Added Input methods to check if cursor is over a selectable actor or a vanilla window, "Deselect bone on click void" uses them.
Added an option to disable "Deselect bone on click void".

The PR should be ready now.

@emyxiv
Copy link
Contributor Author

emyxiv commented Dec 8, 2022

Added a new config to allow deselect when mouse is over an actor, as it becomes tricky with extreme zoom and the "hover target" isn't accurate.

@emyxiv emyxiv force-pushed the mouse-input branch 3 times, most recently from d9a7c8e to ee2ba11 Compare December 10, 2022 01:21
Ktisis/Interface/Input.cs Outdated Show resolved Hide resolved
Ktisis/Interface/Input.cs Outdated Show resolved Hide resolved
@emyxiv emyxiv force-pushed the mouse-input branch 3 times, most recently from 9a95799 to 5bd728b Compare December 22, 2022 21:40
@emyxiv
Copy link
Contributor Author

emyxiv commented Dec 22, 2022

Fixup several commits to clean-up git history.

Fixed the bugs mentioned by @Fayti1703.
Thanks to Ashley for the testing on this PR.

@Fayti1703 Fayti1703 added the has-conflicts PR has conflicts that require rebasing label Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-conflicts PR has conflicts that require rebasing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants