scottcheng / cropit

A "customizable crop and zoom" jQuery plugin.
http://scottcheng.github.io/cropit/
MIT License
874 stars 305 forks source link

JQuery 3.0.0 Compatibility #197

Open josemaldo opened 8 years ago

josemaldo commented 8 years ago

It seems to be a problem with the pluggin and the latest version of JQuery. No prolems with version 2 and below.

Hope it helps :)

cropit-jquery-3 0 0

denvaar commented 8 years ago

I am also seeing this issue. A very hacky hack around it is to declare props as an array right above the line where the error is happening.

Right above

_jquery2['default'].event.props.push('dataTransfer');

put this:

    _jquery2['default'].event.props = [];
exos commented 7 years ago

+1

nobels commented 7 years ago

please fix compatibility problem thanks :-)

benallfree commented 7 years ago

_jquery2['default'].event.props = [];

This is not a good suggestion and requires maintaining your own version of this plugin.

Instead, use jQuery Migrate until the code in this lib is fixed.

charles-passille-smartnets commented 7 years ago

@benallfree the real issue is that this should be done as a fix and pull request.

@denvaar thanks for the solution.

@scottcheng can we see this included in future versions? Thank you.

mlunoe commented 6 years ago

Hitting the same issue. Is there an update to this?

FahriDevZ commented 6 years ago

241 seems to work for me.