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

SVG support extension #12

Open
4 of 5 tasks
edemaine opened this issue Jul 23, 2019 · 0 comments
Open
4 of 5 tasks

SVG support extension #12

edemaine opened this issue Jul 23, 2019 · 0 comments
Labels

Comments

@edemaine
Copy link
Collaborator

edemaine commented Jul 23, 2019

Here are some incompatibilities importing SVG that I've bumped into. Help implementing them would be appreciated; else I will work on them as time allows.

  • Paths do not support z/Z which closes the path (common in Inkscape, for example)
  • Global <style> is not supported (and warned), but common with Adobe Illustrator output.
  • Groups (<g>) with transforms are not supported (but transforms on rendering elements are supported). Sadly not dealt with by THREE.SVGLoader. I had to work around this in https://erikdemaine.org/fonts/maze/ (see source).
  • Objects in <defs> are treated as regular objects. This is annoying because Inkscape tends to make an invisible <rect> in there, which causes a warning (because it has no stroke). Probably same issue with <symbol>.
  • <use> will not expand <defs>/<symbol>s. This is probably less important than the previous item, but I've worked around it many times, so would love to have it.
@edemaine edemaine added the svg label Jul 23, 2019
edemaine added a commit that referenced this issue Jul 25, 2019
Add support for global <style>, as in Illustrator's SVG output (#12)
edemaine added a commit that referenced this issue Jul 25, 2019
Add support for 'z' command in SVG paths (#12)
edemaine added a commit that referenced this issue Dec 2, 2020
Combine transforms from ancestor elements, not just drawing element,
to support e.g. `<g transform>`
edemaine added a commit that referenced this issue Dec 2, 2020
Combine transforms from ancestor elements, not just drawing element,
to support e.g. `<g transform>`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant