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

Handle Autoplayed Video & Audio, Footnotes & Citations #202

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Handle Autoplayed Video & Audio, Footnotes & Citations #202

wants to merge 6 commits into from

Conversation

doakey3
Copy link

@doakey3 doakey3 commented Nov 13, 2017

This is a solution for issues 200 & 201

Autoplayed Media

Video and audio elements with the 'autoplay' attribute will start playing media only when their parent slide is the current slide.

Unfortunately, getting the same behavior with animated gifs would require using other javascript libraries. It can be done, but will take more work to implement.

Footnotes & Citations

Footnote and citation links will change the current slide number to the slide containing the reference target.

doakey3 added 5 commits November 12, 2017 22:19
This update required the use of the
autoplayed media (video & audio) was also fixed (going in and out of
expose mode now behaves as expected).
@doakey3
Copy link
Author

doakey3 commented Nov 18, 2017

I have updated this fix to include stopping animated GIFs from playing until the parent slide is active. This required use of the Gifffer library. There are a couple gotchas with this update:

All images whose src ends with .gif, or base64 images whose "alt" attribute ends with .gif will be changed by the Gifffer function into a button and a canvas element. This may be confusing for someone making a .css file trying to get GIFs positioned a certain way, but not seeing results (because they've been converted to a different tagname). It may be necessary to edit the default .css file and add a comment to clear this confusion.

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.

1 participant