revery-ui / revery

:zap: Native, high-performance, cross-platform desktop apps - built with Reason!
https://www.outrunlabs.com/revery/
MIT License
8.07k stars 196 forks source link

SVG drawing mangled if OS locale does use a different floating point presentation #1085

Closed TheSamsa closed 3 years ago

TheSamsa commented 3 years ago

This issue originated in the oni2 repo. After the Update to the SVG Icons in the sidebar two icons where drawn very odd: 2021-05-31-090724_75x186_scrot

I dug a little bit into the code and discovered there is probably nothing oni2 can change, so I reopened the issue here. It might even a problem with Skia, but since my OCaml is kind of non existant I can't tell you for sure.

zbaylin commented 3 years ago

Hey @TheSamsa -- thanks for pointing this out! I think I may have found the source of this bug in Skia -- are you able to set your esy-skia resolution to 5264b0f in Onivim and see if that fixes the bug?

TheSamsa commented 3 years ago

Hey @zbaylin this did work! I actually thought I found corresponding Issues in Skia (https://bugs.chromium.org/p/skia/issues/detail?id=11132&q=svg%20locale&can=2) and (https://bugs.chromium.org/p/skia/issues/detail?id=11794&q=svg%20locale&can=2) But your solution solved my issue anyways.

zbaylin commented 3 years ago

Awesome @TheSamsa! Glad to hear it. I opened a PR to our Skia fork to hopefully fix this: revery-ui/esy-skia#37

zbaylin commented 3 years ago

Fixed by #1086