Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Float to Double conversion bug for Radius values on Android #2462

Closed
gorkemogut opened this issue Sep 20, 2024 · 5 comments · Fixed by #2415
Closed

Float to Double conversion bug for Radius values on Android #2462

gorkemogut opened this issue Sep 20, 2024 · 5 comments · Fixed by #2415
Assignees
Labels
Missing repro This issue need minimum repro scenario

Comments

@gorkemogut
Copy link

Description

Start getting error on radius styling for objects on Android after updating package from 15.6.0 to 15.7.1.

When we set radius on a Svg object through styling like;

<Svg style={{
        width: 80,
        height: 80,
        borderRadius: 40
    }}></Svg>

Android crashes;
Screenshot 2024-09-20 at 09 16 25

Steps to reproduce

  1. Create object
  2. Give radius from styles on it
  3. Conversion warning and crashes

Snack or a link to a repository

https://github.com/software-mansion/react-native-svg/blob/main/apps/examples/index.tsx

SVG version

15.7.1

React Native version

0.75.3

Platforms

Android

JavaScript runtime

Hermes

Workflow

React Native

Architecture

Paper (Old Architecture)

Build type

Debug app & dev bundle

Device

Android emulator

Device model

Pixel 8 Pro API 34

Acknowledgements

Yes

@github-actions github-actions bot added the Missing repro This issue need minimum repro scenario label Sep 20, 2024
Copy link

Hey! 👋

The issue doesn't seem to contain a minimal reproduction.

Could you provide a snack or a link to a GitHub repository under your username that reproduces the problem?

@gorkemogut
Copy link
Author

Workaround: Using a View as wrapper and giving radius to the View

@bahadiraraz
Copy link

same issue

1 similar comment
@mkondurPPGT
Copy link

same issue

@jakex7 jakex7 self-assigned this Sep 24, 2024
@BertLammers
Copy link

Having the same issue after updating to react-native 0.75.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing repro This issue need minimum repro scenario
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants