hnvn / flutter_image_cropper

A Flutter plugin for Android and iOS supports cropping images
971 stars 374 forks source link

Added size parameter which should result in original scale crops. #400

Open jimmyff opened 1 year ago

jimmyff commented 1 year ago

Added the size parameter set to original as described in http://foliotek.github.io/Croppie/#documentation to get full size crops rather than crops sized at viewport scale.

I've moved on from this plugin as the web implementation doesn't seem to be at the same level as ios & android.

https://github.com/hnvn/flutter_image_cropper/issues/399#issuecomment-1433963244

c-seeger commented 1 year ago

@hnvn let's test if this fix works for #399 and #398

c-seeger commented 1 year ago

@jimmyff just curious what do you use instead?

jimmyff commented 1 year ago

It should work according to the docs but it still was giving me crops too small. I opened an issue at croppies end: https://github.com/Foliotek/Croppie/issues/786 (although Croppie seems like an unmaintained project)

@jimmyff just curious what do you use instead?

As well as the crop size bug I was also getting weird UI issues with this plugin web integration so I'm currently using package:image_cropping. I'm still having issues with that one so I will probably try something else later this week.

It's a shame this plugin doesn't nail the web cropping as the iOS and Android integrations are great.

c-seeger commented 1 year ago

@hnvn maybe switching to a different library could make sense here?

hnvn commented 1 year ago

@jimmyff , @c-seeger , any suggestion for new web lib? I don't have experiences on Web development.

jimmyff commented 1 year ago

Maybe replacing croppie is too drastic and the integration issues could be fixed? It seems like the implmentation is almost there but for my particular purpose on this occassion I was hoping for something working off the shelf.

A quick look on gitthub and https://github.com/fengyuanchen/cropperjs is the most popular project by some margin. It seems very active with a v2 branch currently in alpha.

c-seeger commented 1 year ago

Hm not sure, if in the long term it is easier to maintain since the library is better maintained it may make thinks easier in the long run and brings web to the same level as mobile integrations are.

cropperjs looks like a good choice saw an issue opened last week and a reply from the developer. While Croppie it looks like it's unmaintained.

Painwraith commented 1 year ago

I am facing these exact issues, going to attempt to integrate cropperJS to my Flutter web project. Real shame this isn't maintained. update: Use this for web https://pub.dev/packages/crop_your_image

lucasoares commented 11 months ago

Maybe replacing croppie is too drastic and the integration issues could be fixed? It seems like the implmentation is almost there but for my particular purpose on this occassion I was hoping for something working off the shelf.

A quick look on gitthub and https://github.com/fengyuanchen/cropperjs is the most popular project by some margin. It seems very active with a v2 branch currently in alpha.

PLEASE!!! Croppie is so bad right now