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

Minor interaction bug #21

Open
mindplay-dk opened this issue Oct 18, 2017 · 1 comment
Open

Minor interaction bug #21

mindplay-dk opened this issue Oct 18, 2017 · 1 comment

Comments

@mindplay-dk
Copy link

I've spent two weeks trying to implement the interaction logic for a cropper from scratch, ended up much where you are, and finally decided to just reference your implementation, which is the first and only web-based cropper I've seen that doesn't have any "weird" behavior.

Well, almost. I discovered just one minor interaction bug:

drag-bug

Here I'm dragging the north edge quickly towards north - notice what happens to the bottom edge of the frame? I think you're trying to reject this in calcPosition() with return false, but it happens one frame too late, after the change has already happened. To reproduce this issue, drag quickly - dragging slowly typically causes the bottom edge to be off only by one or two pixels, you might not notice.

I have no idea how to fix this, and suspect it'll require an entirely different design, which is where I've ended up dozens of times while trying to implement this myself. But at this point, I've resigned myself to the attitude of "meh, good enough" - so I completely understand if you don't want to fix this, and, either way, good job! Definitely the closest thing I've seen to a perfect cropper in JS. So thanks! :-)

@flyfloor
Copy link
Owner

@mindplay-dk I have ben here, and thought about redesign it. under this solution, I have no idea how to fix this too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants