Skip to content

Conversation

RicardoNiepel
Copy link

There are currently multiple bugs inside the code base for the SVG support:

  • the PNG DataURL is also assigned to the SVG version
    • written SVG file in the media folder isn't a SVG, it's the PNG - thus Powerpoint cannot load it correctly
  • the mime type for SVG is wrong

This PR fixes all - now the real SVG is used inside the pptx.
You can easily test it with 'pptxgenjs-demo.html': just scale the SVGs before and after this PR.

@mmayer1
Copy link

mmayer1 commented May 23, 2019

This fix works great for me! Looking forward to having a new release with this merged

This was referenced Aug 15, 2019
@gitbrent gitbrent self-assigned this Sep 4, 2019
@gitbrent gitbrent added this to the 3.0.0 milestone Sep 4, 2019
@gitbrent
Copy link
Owner

Thanks @RicardoNiepel !

Issue is fixed in Version 3.0

Previous version:

Screen Shot 2019-09-23 at 23 02 09

Version 3.0:

Screen Shot 2019-09-23 at 23 02 01

@gitbrent gitbrent closed this Sep 24, 2019
gitbrent added a commit that referenced this pull request Sep 24, 2019
@gitbrent gitbrent mentioned this pull request Sep 24, 2019
gitbrent added a commit that referenced this pull request Sep 24, 2019
Also Pull #528 was the main fix here
ericrockey pushed a commit to WeTransfer/PptxGenJS that referenced this pull request Mar 24, 2020
ericrockey pushed a commit to WeTransfer/PptxGenJS that referenced this pull request Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants