search

Chronostasys / Blazor.Cropper

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

Fix double conversion when using another culture than English #32

Closed jro1988 closed 2 years ago

jro1988 commented 2 years ago

Hi,

First of all, thank you for your library. This PR fixes a conversion problem when the user's browser is set to another culture which uses a comma instead of a dot for the decimal part of a number. Because of that, the browser could not interpret the CSS values correctly and the cropper was not working.

Capture d’écran 2021-10-20 à 14 27 49 Capture d’écran 2021-10-20 à 14 28 19

I used the FormattableString.Invariant method to fix this problem.

Chronostasys commented 2 years ago

Thanks very much for the contribution. I will release a new package as soon as I merge it