Background video still resets in 5.1.0 on fragment

Sieboldianus commented 5 months ago

I updated to revealjs 5.1.0 in hope the video reset problem would be gone (#2882), but this is not the case.

See the example slide here.

This is the code for the slide (partly markdown, with jekyll reveal-js):

<div style="position:absolute; margin-left: auto; margin-right: auto; top: 50px; left: 0; bottom: 0;" class="fragment">      
<p style="text-align:left;color:#00FEFF">
4 Facets:<br>
Location (Where?)<br>  
User origin (Who?)<br>
Tags (What?)<br>
Time (When?)
</p> 
</div>

<!-- .slide: data-background-video="https://wwwpub.zih.tu-dresden.de/~s7398234/slide_assets/2017_FlickrEuropeAni.webm" data-background-video-loop data-background-video-muted -->

That said.. thank you so much for RevealJS; overall it is a blessing to have!

hakimel commented 5 months ago

When I test this in the latest reveal.js version locally, background video isn't restarting when fragments are shown. This is the content I used to test:

<div class="slides">
  <section>Slide 1</section>
  <section data-background-video="https://wwwpub.zih.tu-dresden.de/~s7398234/slide_assets/2017_FlickrEuropeAni.webm" data-background-video-loop data-background-video-mute>
    Slide 2
    <div class="fragment">Fragment 1</div>
    <div class="fragment">Fragment 2</div>
  </section>
</div>

I do see the video restarting in your live demo. Not sure what the difference is.

Sieboldianus commented 5 months ago

Thank you for testing @hakimel , I will investigate this further and update here!

Sieboldianus commented 1 month ago

I need to debug this further, but I think it could be related to the note-plugin, see https://github.com/hakimel/reveal.js/pull/3630#issue-2321761306 - [edit] Merging the fix did not solve the video restarting problem on my side..

hakimel / reveal.js

Background video still resets in 5.1.0 on fragment #3633