Skip to content

Conversation

@vyrnix9582046173
Copy link
Contributor

This PR fixes a studio bug where when you click and hold transform while moving mouse outside of window, let go of click, moving cursor in viewport still is in transform mode.

@vyrnix9582046173 vyrnix9582046173 mentioned this pull request May 17, 2019
@vyrnix9582046173
Copy link
Contributor Author

there is a hack here with transformEnable that disables the transform movement when the mouse enters the UI, but that does not cover 100% cases: this bug still occurs when dragging mouse on the left side where there is no UI:

leftCapture

@vyrnix9582046173
Copy link
Contributor Author

vyrnix9582046173 commented May 17, 2019

The next step is detecting when the users mouse is no longer moving inside of engine-render https://github.com/exokitxr/exokit/pull/1065/files#diff-818c6db2ba21b9def5bafb0e208c1111R215

@avaer
Copy link
Member

avaer commented May 17, 2019

Is there a way to get mouseup when not in the window? I figure GLFW must have that capability 🤔

@vyrnix9582046173
Copy link
Contributor Author

Not finding anything in a quick search, should dragging outside of the window be allowed behavior here?

@avaer
Copy link
Member

avaer commented May 17, 2019

Yes, I think so. We just need to handle the mouse moving up.

@vyrnix9582046173 vyrnix9582046173 modified the milestone: 1.0 May 17, 2019
@vyrnix9582046173
Copy link
Contributor Author

mouseup is now handled on window, which fixes this bug.

@vyrnix9582046173 vyrnix9582046173 requested a review from avaer May 17, 2019 22:53
@vyrnix9582046173 vyrnix9582046173 changed the title [WIP] Studio transform bug Studio transform bug May 18, 2019
@avaer avaer merged commit 32ec4a3 into studio May 18, 2019
@avaer avaer deleted the transform-state branch May 18, 2019 03:02
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 this pull request may close these issues.

3 participants