YPImagePicker icon indicating copy to clipboard operation
YPImagePicker copied to clipboard

If onlySquare is enabled, execute fitImage on display.

Open hanawat opened this issue 1 year ago • 4 comments

#776

To fix above issue.

If onlySquare is enabled when displaying an Asset, call fitImage to crop it to a square.

You can see how it works in the sample apps in the feature/demo branch. https://github.com/hanawat/YPImagePicker/tree/feature/demo

Before After
スクリーンショット 2022-12-15 21 33 07 スクリーンショット 2022-12-15 21 33 21

hanawat avatar Dec 15 '22 12:12 hanawat

With this fix, the selected image view can't be moved up when scrolling. Can you take a look?

ramboli1986 avatar Feb 15 '23 22:02 ramboli1986

@keremcesme Thank you for your review. Videos in the library also used fitImage.

hanawat avatar Feb 17 '23 07:02 hanawat

@ramboli1986 Did you run that in the sample app? I checked and it works fine in the sample app.

hanawat avatar Feb 17 '23 07:02 hanawat

@ramboli1986 Did you run that in the sample app? I checked and it works fine in the sample app.

Nvm, I can't repo it any more. Thanks for the fix.

ramboli1986 avatar Feb 22 '23 01:02 ramboli1986