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

#119 Fix SVG export #135

Merged
merged 1 commit into from
Feb 26, 2023
Merged

#119 Fix SVG export #135

merged 1 commit into from
Feb 26, 2023

Conversation

matteotrapani
Copy link

@matteotrapani matteotrapani commented Feb 21, 2023

Added specific handling of SVG extension to fix issue #119

I could have added a strategy or something to avoid hard-coded strings, but I've seen that there is not a huge usage of these patterns in the code so I didn't want to make a huge PR.

If you have any suggestion please feel free to make them

Furthermore I noticed that on my local PC there is some issues with the tests because only the ones I created were working properly. I fear that it is something related to encoding or culture. Let me know if I can fix it somehow.

Thanks

Added specific handling of SVG extension
@MaxMelcher MaxMelcher merged commit f1901b7 into MaxMelcher:main Feb 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants