-
Notifications
You must be signed in to change notification settings - Fork 21
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
issue with cropper source #14
Comments
@rinoldsimon could you send me what options you are setting prior to changing the source? There may be some we are missing. There are some properties where we tear down and construct a new Cropper instance completely, so some of those options may be getting lost if we aren't copying things over properly. |
hbs
I'm using
In this scenario According the |
Right, we are calling I wonder if we're just missing passing additional options after a replace or something. I can dig in a bit more later this evening. |
Think this is not the solution According to the |
is the addon uses the latest version of |
Thanks for digging in more, @rinoldsimon. Could you try using the branch at https://github.com/danielthall/ember-cropperjs/tree/replace-hasSameSize-fix and let me know if this resolves the issue for you? You'll just need to add the following to your {{#image-cropper
replaceHasSameSize=true
.
. I have an open issue for upgrading to the latest version of |
I forgot to add this rule.
adding the above rule fixes the issue |
@rinoldsimon glad you got it figured out! |
when i upload a different image and update the {{#cropper-image
source=*thelatestuploadedimageurl*
... }}. the source changes but the previous options are not getting set to the new image.The text was updated successfully, but these errors were encountered: