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.

Sign up for GitHub

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

Jump to bottom

SVG import ignores rectangle radius when only one side is set #1863

Open
sasensi opened this issue Oct 25, 2020 · 0 comments · May be fixed by #1866
Open

SVG import ignores rectangle radius when only one side is set #1863

sasensi opened this issue Oct 25, 2020 · 0 comments · May be fixed by #1866
Labels
cat: svg status: PR proposed an active pull request should fix this issue type: bug

Comments

@sasensi
Copy link
Contributor

sasensi commented Oct 25, 2020

Description/Steps to reproduce

  • import a <rect> with rx set but not ry (which is optional in SVG spec)
  • see that the imported shape radius is null
    Capture d’écran 2020-10-25 153834

Link to reproduction test-case

The bug is reproduced in this sketch.

Expected result

The imported shape radius should be set.
Capture d’écran 2020-10-25 153849
sasensi added a commit to sasensi/paper.js that referenced this issue Nov 4, 2020
@sasensi
Fix: svg import ignores rectangle radius when only one side is set.
50b9b6b 
This follows the SVG spec in which only one side can be set and set the other
side to the same value in this case.
This also adds support for percent values.

Fixes paperjs#1863
@sasensi sasensi linked a pull request Nov 4, 2020 that will close this issue
Open
2 tasks
sasensi added a commit to sasensi/paper.js that referenced this issue Nov 4, 2020
@sasensi
Fix: svg import ignores rectangle radius when only one side is set.
216412d 
This follows the SVG spec in which only one side can be set and set the other
side to the same value in this case.
This also adds support for percent values.

Fixes paperjs#1863
@sasensi sasensi added the status: PR proposed an active pull request should fix this issue label Nov 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cat: svg status: PR proposed an active pull request should fix this issue type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: svg import ignores rectangle radius when only one side is set. sasensi/paper.js
1 participant
@sasensi