These are chat archives for matiasgagliano/guillotine

5th
Oct 2016
Jonathon Terry
@Maverick1872
Oct 05 2016 17:37
@matiasgagliano i'm running into an issue with subsequent initializations not taking a new height/width for the crop size. the following gist should include all the necessary code https://gist.github.com/Maverick1872/fc6e48671c0fa7b4104a34611b5f9bf0.
as you can see I remove instances in two locations, once at the modal dialog close action, and again before initialization if there is still one (someone clicks the x-button on the modal). I have verified with console.log statements that the height and width is getting passed but they don't seem to be updating.
Jonathon Terry
@Maverick1872
Oct 05 2016 19:51
nvm looks like it was a stray quote on line 27. if you see anything wrong or if you suggest I do something a different way though I would be more than happy to hear
Matías Gagliano
@matiasgagliano
Oct 05 2016 19:59
At first sight the picture.on('load', ... part seems to be fine. Could you provide a jsFiddle or CodePen that shows the issue?
Jonathon Terry
@Maverick1872
Oct 05 2016 20:25
Ahh it's been fixed now. One of those issues that just seems to sort itself out, aren't they the best?
I do appreciate it though and it seems to be an awesome plugin!
Matías Gagliano
@matiasgagliano
Oct 05 2016 20:27
Indeed they are, as long as they don't come back!
Thanks, I'm glad it all works now.