From 5783c8a93b72d232b21bf3d7da7aa676306ef763 Mon Sep 17 00:00:00 2001
From: itmammoth <itmammoth@gmail.com>
Date: Sat, 8 Feb 2025 13:30:15 +0900
Subject: [PATCH 1/3] Support `video` tag with `poster` attribute

---
 markdownify/__init__.py   | 13 +++++++++++++
 tests/test_conversions.py |  6 ++++++
 2 files changed, 19 insertions(+)

diff --git a/markdownify/__init__.py b/markdownify/__init__.py
index 9e4c99f..a883db0 100644
--- a/markdownify/__init__.py
+++ b/markdownify/__init__.py
@@ -463,6 +463,19 @@ def convert_img(self, el, text, convert_as_inline):
 
         return '![%s](%s%s)' % (alt, src, title_part)
 
+    def convert_video(self, el, text, convert_as_inline):
+        # `text` is a fallback for browsers that do not support video, so it's unused
+        src = el.attrs.get('src', None) or ''
+        poster = el.attrs.get('poster', None) or ''
+        if (convert_as_inline
+                and el.parent.name not in self.options['keep_inline_images_in']):
+            return ''
+        if not poster:
+            return ''
+        if src:
+            return '[![](%s)](%s)' % (poster, src)
+        return '![](%s)' % poster
+
     def convert_list(self, el, text, convert_as_inline):
 
         # Converting a list to inline is undefined.
diff --git a/tests/test_conversions.py b/tests/test_conversions.py
index 1739cb9..0d14c78 100644
--- a/tests/test_conversions.py
+++ b/tests/test_conversions.py
@@ -242,6 +242,12 @@ def test_img():
     assert md('<img src="/path/to/img.jpg" alt="Alt text" />') == '![Alt text](/path/to/img.jpg)'
 
 
+def test_video():
+    assert md('<video src="/path/to/video.mp4" poster="/path/to/img.jpg" />') == '[![](/path/to/img.jpg)](/path/to/video.mp4)'
+    assert md('<video poster="/path/to/img.jpg" />') == '![](/path/to/img.jpg)'
+    assert md('<video poster="/path/to/img.jpg">Fallback text</video>') == '![](/path/to/img.jpg)'
+
+
 def test_kbd():
     inline_tests('kbd', '`')
 

From d51b55336c8d32c04643db57e59eb61ecef87bf7 Mon Sep 17 00:00:00 2001
From: itmammoth <itmammoth@gmail.com>
Date: Wed, 26 Feb 2025 11:49:55 +0900
Subject: [PATCH 2/3] Enhance video conversion handling and update tests

https://github.com/matthewwithanm/python-markdownify/pull/189#discussion_r1957910479
---
 markdownify/__init__.py   | 23 ++++++++++++++---------
 tests/test_conversions.py |  8 +++++---
 2 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/markdownify/__init__.py b/markdownify/__init__.py
index 4d7ac50..50515c7 100644
--- a/markdownify/__init__.py
+++ b/markdownify/__init__.py
@@ -538,18 +538,23 @@ def convert_img(self, el, text, parent_tags):
 
         return '![%s](%s%s)' % (alt, src, title_part)
 
-    def convert_video(self, el, text, convert_as_inline):
-        # `text` is a fallback for browsers that do not support video, so it's unused
+    def convert_video(self, el, text, parent_tags):
+        if ('_inline' in parent_tags
+                and el.parent.name not in self.options['keep_inline_images_in']):
+            return text
         src = el.attrs.get('src', None) or ''
+        if not src:
+            sources = el.find_all('source', attrs={'src': True})
+            if sources:
+                src = sources[0]['src']
         poster = el.attrs.get('poster', None) or ''
-        if (convert_as_inline
-                and el.parent.name not in self.options['keep_inline_images_in']):
-            return ''
-        if not poster:
-            return ''
+        if src and poster:
+            return '[![%s](%s)](%s)' % (text, poster, src)
         if src:
-            return '[![](%s)](%s)' % (poster, src)
-        return '![](%s)' % poster
+            return '[%s](%s)' % (text, src)
+        if poster:
+            return '![%s](%s)' % (text, poster)
+        return text
 
     def convert_list(self, el, text, parent_tags):
 
diff --git a/tests/test_conversions.py b/tests/test_conversions.py
index a6abd6d..b406a9f 100644
--- a/tests/test_conversions.py
+++ b/tests/test_conversions.py
@@ -244,9 +244,11 @@ def test_img():
 
 
 def test_video():
-    assert md('<video src="/path/to/video.mp4" poster="/path/to/img.jpg" />') == '[![](/path/to/img.jpg)](/path/to/video.mp4)'
-    assert md('<video poster="/path/to/img.jpg" />') == '![](/path/to/img.jpg)'
-    assert md('<video poster="/path/to/img.jpg">Fallback text</video>') == '![](/path/to/img.jpg)'
+    assert md('<video src="/path/to/video.mp4" poster="/path/to/img.jpg">text</video>') == '[![text](/path/to/img.jpg)](/path/to/video.mp4)'
+    assert md('<video src="/path/to/video.mp4">text</video>') == '[text](/path/to/video.mp4)'
+    assert md('<video><source src="/path/to/video.mp4"/>text</video>') == '[text](/path/to/video.mp4)'
+    assert md('<video poster="/path/to/img.jpg">text</video>') == '![text](/path/to/img.jpg)'
+    assert md('<video>text</video>') == 'text'
 
 
 def test_kbd():

From e98c86ffb68c84f21902f2396b1af17322d6ffdc Mon Sep 17 00:00:00 2001
From: itmammoth <itmammoth@gmail.com>
Date: Wed, 26 Feb 2025 11:57:28 +0900
Subject: [PATCH 3/3] Fix source attribute retrieval to ensure fallback to
 empty string

---
 markdownify/__init__.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/markdownify/__init__.py b/markdownify/__init__.py
index 50515c7..3763a49 100644
--- a/markdownify/__init__.py
+++ b/markdownify/__init__.py
@@ -546,7 +546,7 @@ def convert_video(self, el, text, parent_tags):
         if not src:
             sources = el.find_all('source', attrs={'src': True})
             if sources:
-                src = sources[0]['src']
+                src = sources[0].attrs.get('src', None) or ''
         poster = el.attrs.get('poster', None) or ''
         if src and poster:
             return '[![%s](%s)](%s)' % (text, poster, src)