Merge branch 'main' into pre-commit-ci-update-config

.github/ISSUE_TEMPLATE/bug.yml

@@ -122,7 +122,7 @@ body:
       This will be automatically formatted into code, so no need for backticks.
     placeholder: |
       from manim import *
-      from manim_slides.slide import Slide
+      from manim_slides import Slide
 
 
       class MWE(Slide):

.python-version

@@ -1 +1 @@
-3.11.8
+3.11

.readthedocs.yaml

@@ -2,7 +2,7 @@ version: 2
 build:
   os: ubuntu-22.04
   tools:
-    python: '3.10'
+    python: '3.11'
   apt_packages:
   - libpango1.0-dev
   - ffmpeg

CHANGELOG.md

@@ -10,6 +10,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 (unreleased)=
 ## [Unreleased](https://github.com/jeertmans/manim-slides/compare/v5.3.0...HEAD)
 
+(unreleased-fixed)=
+### Fixed
+
+- Fixed HTML template to avoid missing slides when exporting with `--one-file`.
+  [@Rapsssito](https://github.com/Rapsssito) [#515](https://github.com/jeertmans/manim-slides/pull/515)
+
 (v5.3.0)=
 ## [v5.3.0](https://github.com/jeertmans/manim-slides/compare/v5.2.0...v5.3.0)
 

docs/source/_static/template.html

@@ -316,26 +316,47 @@
       });
 
       {% if one_file %}
-        // Fix found by @t-fritsch and @Rapsssito on GitHub
-        // see: https://github.com/hakimel/reveal.js/discussions/3362#discussioncomment-11733074.
-        function fixBase64VideoBackground(event) {
-          // Analyze all slides backgrounds
-          for (const slide of Reveal.getBackgroundsElement().querySelectorAll('.slide-background')) {
-            // Get the slide video and its sources for each background
-            const video = slide.querySelector('video');
-            const sources = video.querySelectorAll('source');
-            // Update the source of the video
-            sources.forEach((source, i) => {
-              const src = source.getAttribute('src');
-              if(src.match(/^data:video.*;base64$/)) {
-                const nextSrc = sources[i+1]?.getAttribute('src');
-                video.setAttribute('src', `${src},${nextSrc}`);
+      // Fix found by @t-fritsch and @Rapsssito on GitHub
+      // see: https://github.com/hakimel/reveal.js/discussions/3362#discussioncomment-11733074.
+      function setVideoBase64(video) {
+        const sources = video.querySelectorAll('source');
+        // Update the source of the video
+        sources.forEach((source, i) => {
+          const src = source.getAttribute('src');
+          if(src.match(/^data:video.*;base64$/)) {
+            const nextSrc = sources[i+1]?.getAttribute('src');
+            video.setAttribute('src', `${src},${nextSrc}`);
+          }
+        });
+      }
+
+      function fixBase64VideoBackground(event) {
+        // Analyze all slides backgrounds
+        for (const slide of Reveal.getBackgroundsElement().querySelectorAll('.slide-background')) {
+          // Get the slide video and its sources for each background
+          const video = slide.querySelector('video');
+          if (video) {
+            setVideoBase64(video);
+          } else {
+            // Listen to the creation of the video element
+            const observer = new MutationObserver((mutationsList) => {
+              for (const mutation of mutationsList) {
+                if (mutation.type === 'childList') {
+                  for (const addedNode of mutation.addedNodes) {
+                    if (addedNode.tagName === 'VIDEO') {
+                      setVideoBase64(addedNode);
+                      observer.disconnect(); // Stop observing once the video is handled
+                    }
+                  }
+                }
               }
             });
+            observer.observe(slide, { childList: true, subtree: true });
           }
         }
-        // Setup base64 videos
-        Reveal.on( 'ready', fixBase64VideoBackground );
+      }
+      // Setup base64 videos
+      Reveal.on( 'ready', fixBase64VideoBackground );
       {% endif %}
     </script>
 

manim_slides/templates/revealjs.html

@@ -323,20 +323,41 @@
       {% if one_file -%}
       // Fix found by @t-fritsch and @Rapsssito on GitHub
       // see: https://github.com/hakimel/reveal.js/discussions/3362#discussioncomment-11733074.
+      function setVideoBase64(video) {
+        const sources = video.querySelectorAll('source');
+        // Update the source of the video
+        sources.forEach((source, i) => {
+          const src = source.getAttribute('src');
+          if(src.match(/^data:video.*;base64$/)) {
+            const nextSrc = sources[i+1]?.getAttribute('src');
+            video.setAttribute('src', `${src},${nextSrc}`);
+          }
+        });
+      }
+
       function fixBase64VideoBackground(event) {
         // Analyze all slides backgrounds
         for (const slide of Reveal.getBackgroundsElement().querySelectorAll('.slide-background')) {
           // Get the slide video and its sources for each background
           const video = slide.querySelector('video');
-          const sources = video.querySelectorAll('source');
-          // Update the source of the video
-          sources.forEach((source, i) => {
-            const src = source.getAttribute('src');
-            if(src.match(/^data:video.*;base64$/)) {
-              const nextSrc = sources[i+1]?.getAttribute('src');
-              video.setAttribute('src', `${src},${nextSrc}`);
-            }
-          });
+          if (video) {
+            setVideoBase64(video);
+          } else {
+            // Listen to the creation of the video element
+            const observer = new MutationObserver((mutationsList) => {
+              for (const mutation of mutationsList) {
+                if (mutation.type === 'childList') {
+                  for (const addedNode of mutation.addedNodes) {
+                    if (addedNode.tagName === 'VIDEO') {
+                      setVideoBase64(addedNode);
+                      observer.disconnect(); // Stop observing once the video is handled
+                    }
+                  }
+                }
+              }
+            });
+            observer.observe(slide, { childList: true, subtree: true });
+          }
         }
       }
       // Setup base64 videos

pyproject.toml

@@ -49,6 +49,7 @@ docs = [
   "myst-parser>=2.0.0",
   "nbsphinx>=0.9.2",
   "pandoc>=2.3",
+  "pygments<2.19",  # See: https://github.com/ManimCommunity/manim/issues/4104
   "sphinx>=7.0.1",
   "sphinxcontrib-programoutput>=0.18",
   "sphinx-design>=0.6.1",
@@ -60,7 +61,7 @@ full = [
   "manim-slides[magic,manim,sphinx-directive]",
 ]
 magic = ["manim-slides[manim]", "ipython>=8.12.2"]
-manim = ["manim>=0.18.0"]
+manim = ["manim>=0.17"]
 manimgl = ["manimgl>=1.7.2"]
 pyqt6 = ["pyqt6>=6.7.0"]
 pyqt6-full = ["manim-slides[full,pyqt6]"]
@@ -136,6 +137,13 @@ replace = '''<!-- start changelog -->
 ## [Unreleased](https://github.com/jeertmans/manim-slides/compare/v{new_version}...HEAD)'''
 search = "<!-- start changelog -->"
 
+[[tool.bumpversion.files]]
+filename = "uv.lock"
+replace = '''name = "manim-slides"
+version = "{new_version}"'''
+search = '''name = "manim-slides"
+version = "{current_version}"'''
+
 [tool.codespell]
 builtin = "clear,rare,informal,usage,names,en-GB_to_en-US"
 check-hidden = true