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

horizontal mouse rotate is always on world axes #38

Closed
ssfrr opened this issue Dec 28, 2017 · 4 comments
Closed

horizontal mouse rotate is always on world axes #38

ssfrr opened this issue Dec 28, 2017 · 4 comments

Comments

@ssfrr
Copy link
Contributor

ssfrr commented Dec 28, 2017

it seems that vertically rotating with the mouse up/down acts on the camera's coordinate system, but horizontal rotation (mouse left-right) is rotating along the world's axis. To me it seems like it would be more intuitive to have horizontal rotation also be around the camera's "up" axis, but I suppose this could be a subjective preference.

@ssfrr ssfrr changed the title mouse rotate is always on world axes horizontal mouse rotate is always on world axes Dec 28, 2017
@SimonDanisch
Copy link
Member

this might have been fixed by: JuliaGL/GLAbstraction.jl@79cb6e3

@ssfrr
Copy link
Contributor Author

ssfrr commented Dec 28, 2017

just from looking at it, that seems like it would either rotate around (0, 0, 1) or (0, 0, -1) based on the camera angle. seems like it would be an improvement but you'd still have the behavior, e.g. if you're looking down from above and you drag left and right the scene would appear to rotate beneath you like the hand of a clock, rather than rotating relative to your viewpoint. I can play around with it a bit and maybe propose a PR you can try out and see if you like it better.

@SimonDanisch
Copy link
Member

did you see JuliaGL/GLAbstraction.jl#76 ?

@ssfrr
Copy link
Contributor Author

ssfrr commented Dec 28, 2017

Ah, I see. Looks like you’ve tried it and prefer the way it is. I’ll see if I get used to it and if I still like the other way maybe I’ll do a PR to make it configurable somehow

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

No branches or pull requests

2 participants