Boundary should default to size of the container.

jofftiquez / vue-croppie

Vue wrapper for croppie

https://jofftiquez.github.io/vue-croppie/

MIT License

260 stars 42 forks source link

Boundary should default to size of the container. #26

Closed MatthewAry closed 5 years ago

MatthewAry commented 6 years ago

vue-croppie's prop for boundary defaults to 400x400 px when it should default to the size of the container. See: https://foliotek.github.io/Croppie/#boundary

By having a default boundary size behaviors it breaks complex layouts such as those using flex.

jofftiquez commented 6 years ago

@MatthewAry thanks for mentioning this. I'll be working on it.

rdomingue commented 6 years ago

Nothing new on this ?

jofftiquez commented 6 years ago

@rdomingue sorry it's been quite some time. I will update my timeline on this. Been busy in a while now. :( Thank you for your patience.

saeedvz commented 5 years ago

any updates?

jofftiquez commented 5 years ago

Hi all, sorry for the delay, I've been very busy lately. Anyways, the fix is now available in 1.3.13. I removed the default value from the boundary props. You just gotta pass a height tho so the layout won't be a mess. You can see that the demo is now responsive.

Refer to : L#15 L#76