-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
rect bug when parameters are clamped #7
Comments
I guess it's because I used :delta-pos (stores the delta of mouse position from previous function call literally) for some of the handles. https://github.com/baku89/glisp/blob/master/public/lib/math.glisp#L671-L683 It might be better not to calculate new parameters incrementally in some cases. |
Even I refactored the drag function of Thank you for a lot of feedback anyway! |
https://glisp.app/commit:69e6023/?code_url=https%3A%2F%2Fgist.githubusercontent.com%2Fmicuat%2Ff8b1e0001efcf2115bca90dcbee93979%2Fraw%2F9d24021efc890dc514710bd0805c32cca9d69384%2Fclamprectbug.glisp
in this case, when x y are bound by clamp, width and height still moves in an unexpected way (I guess it's trying to compensate values somehow)
The text was updated successfully, but these errors were encountered: