Distance Threshold seemingly not working

SpectralVectors / RightMouseNavigation

Game Engine-style Right Mouse Viewport Navigation

GNU General Public License v3.0

221 stars 18 forks source link

Distance Threshold seemingly not working #22

Closed ettiSurreal closed 5 months ago

ettiSurreal commented 5 months ago

The tooltip says it's how far you have to move the mouse to trigger navigation, but it begins immediately after pressing it, without even moving the mouse. Also creates a slightly annoying flicker when quickly pressed.

SpectralVectors commented 5 months ago

I will take this into account, there are some existing bugs and Blender 4.1 issues that I have to sort out soon. I will let you know if I can figure out why that is happening. Thanks!

ettiSurreal commented 5 months ago

Definitely doesn't seem like a 4.1 breakage, checking it in 4.0.2 and it doesn't work properly there either. it only seems to affect the distance before the context menu can no longer open.

Reset cursor doesn't seem to work at all either.

SpectralVectors commented 5 months ago

Yes, I should be clear: I have a 4.1 issue to sort out:

users have reported Middle Mouse navigation is not working correctly

There are also existing bugs:

keymap registration is bugged
Distance Threshold is bugged (I'm not sure if this ever worked, but I'm also not sure why it didn't)
Apparently reset cursor is also broken - this definitely did work, so it could be the result of an API change

I will let you know when I have an update for any of these issues. Alternatively, you're welcome to submit a pull request if you know of a fix. Thanks!

SpectralVectors commented 5 months ago

@ettiSurreal I was able to fix the Reset Cursor behavior, but the Distance Threshold didn't want to cooperate, so I just removed it. Test release is available here: https://github.com/SpectralVectors/RightMouseNavigation/releases/tag/keymap-fix-release

SpectralVectors commented 5 months ago

I'm marking this issue as closed since the 'Distance Threshold' setting has been removed. Feel free to create another issue if you find another bug, or you want the 'Distance Threshold' to be implemented. Thanks for using the addon!