Skip to content

Extended support of SVG format#4168

Merged
benfry merged 1 commit into
processing:masterfrom
poqudrof:master-origin
Jan 18, 2019
Merged

Extended support of SVG format#4168
benfry merged 1 commit into
processing:masterfrom
poqudrof:master-origin

Conversation

@poqudrof

Copy link
Copy Markdown

Hello Processing maintainers !

I work on a side project called Soby in which I do slideshows based on SVGs, it is based on an older Sozi.
I extended a little bit the support of SVGs and now with Java 8 and stuff like the Base64 decoder I have a rather clean solution.

Here are the good parts : 

  • Support of embedded images in SVGs, linked images are supported but should not be avoided if possible.
  • Text and image locations are identical to SVG visualized in Inkscape.
    When the font works, it fits perfectly.
  • Low modification of current code except for new parsing functions.

Here are the bad parts : 

  • I introduce some code duplication in :
private void loadBase64Image() 
  • I have no idea how to handle text quality in a clean way... :
 public static float TEXT_QUALITY = 1;   

Although is seems 2 is a good «guess» for most applications.

  • Inline styles are not supported.
  • Not all fonts are supported (depending on your OS).
  • Tested only on Inkscape SVGs.

…hould not be avoided if possible.

Support of Text in SVGs (partial). Multiline text are also displayed.

Text limitations :
Inline styles are not supported.
Not all fonts are supported (depending on your OS).
Tested only on Inkscape SVGs.
@poqudrof poqudrof closed this May 4, 2018
@poqudrof

poqudrof commented May 4, 2018

Copy link
Copy Markdown
Author

This is now a library : https://github.com/Rea-lity-Tech/SVGExtended

@jeremydouglass

Copy link
Copy Markdown
Contributor

@poqudrof thanks for sharing this! Consider announcing this on the new forum -- and if you can, please submit it as a confirming library to the PDE Contribution Manager

https://github.com/processing/processing/wiki/Library-Guidelines

@benfry benfry reopened this Jan 18, 2019
@benfry benfry merged commit 73d5cae into processing:master Jan 18, 2019
@clankill3r clankill3r mentioned this pull request Oct 4, 2019
@github-actions

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants