Display:none causes crop to reset about ngimgcrop HOT 6 OPEN

@atomworks I'm taking a look at this myself right now

from ngimgcrop.

Excellent. If I can provide any other information just let me know and I'll see what I can do!

from ngimgcrop.

Ok I think i might have it. I'm not going to do a pull request as I have a slightly mangled / feature diverse repo. HOWEVER, try replacing the following lines at the end of setMaxDimensions in crop-host.js

var center = theArea.getCenterPoint();
theArea.setCenterPoint({x: center.x*ratioNewCurWidth, y: center.y*ratioNewCurHeight});

with

var center = theArea.getCenterPoint();
theArea.setSize({w: theArea.getSize().w * ratioNewCurWidth, h: theArea.getSize().h * ratioNewCurHeight});
theArea.setCenterPoint({x: center.x * ratioNewCurWidth, y: center.y * ratioNewCurHeight});

from ngimgcrop.

Can I assume from your silence that this worked? :)

from ngimgcrop.

Hi Mike, apologies!

I'm running a circuit on tasks for this project and haven't got round to checking this yet. I was expecting to have had a chance by now so apologies for not responding!

I'll take a look at it and get back to you today / tomorrow! Thanks for taking a look in to this, much appreciated!

from ngimgcrop.

+1
got the same issue but in crop-host.js there are not those line of code at the end of setMaxDimensions.
I tried to add the fix you suggest at the end of setMaxDimension but it didn't work.

Edit:
I found out that deleting the code line

elCanvas.prop('width',canvasDims[0]).prop('height',canvasDims[1]).css({'margin-left': -canvasDims[0]/2+'px', 'margin-top': -canvasDims[1]/2+'px'}); var ratioNewCurWidth=ctx.canvas.width/curWidth, ratioNewCurHeight=ctx.canvas.height/curHeight, ratioMin=Math.min(ratioNewCurWidth, ratioNewCurHeight); theArea.setX(theArea.getX()*ratioNewCurWidth); theArea.setY(theArea.getY()*ratioNewCurHeight); theArea.setSize(theArea.getSize()*ratioMin); console.log(theArea.getSize()*ratioMin);

in setMaxDimension, the ngImgCrop plugin works fine.

from ngimgcrop.