search

Chronostasys / Blazor.Cropper

A blazor library provides component to crop image
MIT License
84 stars 19 forks source link

Question: Why can't I get to the left part of this image? #41

Closed hounddog22030 closed 2 years ago

hounddog22030 commented 2 years ago

Been doing some more testing.

Steps To Reproduce:

  1. Edit CropperExample.razor (c#), change from ... to MaxCropedHeight="15000" MaxCropedWidth="15000"
  2. Run Sample Project
  3. Click Choose File
  4. Select File created from https://digital-assets.partshub.com/DVJP/460f3c4f0aba9.jpg
  5. Attempt to drag over to the leftmost part of the image (see above image file in step 4 in a separate browser tab)

Expected Result:

Able to drag to the leftmost part of the image in order to crop it.

Observed Result:

Unable to get to appx left 20% of the image.

This is reproducible from before the PR you accepted from me (commit b112ffecc656b1c95bc36546521da7fb0617b457).

Chronostasys commented 2 years ago

This issue is because the cropper is initially set to the left-top part of the image. And the image above is far too wide that the whole cropper box is not visible, and the image is simply "stucked" by it.
For this issue, you can resolve this problem by setting OffsetX to 1500.

Chronostasys commented 2 years ago

I dig deeper into this issue today, and I thought current design is not good enough. I've got a new idea, and may resolve this issue in the next version.

hounddog22030 commented 2 years ago

Yeah, the X,Y is not correct either if you zoom it. We're going to switch to cropper.js. Thanks for your help and good luck with graduation.

On Thu, May 26, 2022 at 10:24 AM Boxiu Lee @.***> wrote:

I dig deeper into this issue today, and I thought current design is not good enough. I've got a new idea, and may resolve this issue in the next version.

— Reply to this email directly, view it on GitHub https://github.com/Chronostasys/Blazor.Cropper/issues/41#issuecomment-1138635490, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVUT3QS22UM67JFRLSKKMDVL6CTPANCNFSM5W2YKLXA . You are receiving this because you authored the thread.Message ID: @.***>

Chronostasys commented 2 years ago

Thanks. Fixed in 1.2.4.