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

addImage() - Support for animated GIFs? #22

Closed
shashank2104 opened this issue Jan 11, 2017 · 5 comments
Closed

addImage() - Support for animated GIFs? #22

shashank2104 opened this issue Jan 11, 2017 · 5 comments
Assignees

Comments

@shashank2104
Copy link

Hi,

Thank you for the wonderful project. It is indeed very useful.
Issue: I tried adding a gif image to the ppt, it works fine but the animation doesn't. It gets embedded as an image and not a movie. (just copying a gif into a ppt embeds itself as a movie)

Do you think that feature is already available and I'm doing it wrong or is it something that still needs to be implemented?

Can you try that and let me know if it works?

Thanks.

@gitbrent gitbrent self-assigned this Jan 11, 2017
@gitbrent gitbrent changed the title addImage (gif ? ) addImage() - Support for animated GIFs? Jan 11, 2017
@gitbrent
Copy link
Owner

Thanks for the feedback!

I never tried using animated GIFs in a Presentation... let me look into how the images are being encoded/included and see what can be done to support this.

@shashank2104
Copy link
Author

Sure! Thank you.

@shashank2104
Copy link
Author

Hey @gitbrent ,

Any updates on this issue?
Do you think this feature can be added?

Thanks.

@gitbrent
Copy link
Owner

Good news! This can be done... but in only 1 of 2 ways.

If you use Node.js or pre-encode the gif and pass it to addImage() using the data option, it'll render as an animated gif in PPT.

Images added with path: cannot be encoded into GIFs because the method for doing so isn't supported yet (even in Chrome).

Version 1.1.6 will be out soon and will provide this functionality.

gitbrent pushed a commit that referenced this issue Jan 19, 2017
@shashank2104
Copy link
Author

Hey Brent,

I did test it using the demo file and it works absolutely fine. Thanks a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants