From 5316f8e0bdb662ef1c378043a85ce942ed646c83 Mon Sep 17 00:00:00 2001
From: Navin Karkera <navin@opencraft.com>
Date: Thu, 6 Mar 2025 11:46:17 +0530
Subject: [PATCH 1/2] feat: preserve image alt text while converting html to
 text

---
 xmodule/annotator_mixin.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/xmodule/annotator_mixin.py b/xmodule/annotator_mixin.py
index 8e7617e1cabd..b3aaefa31c3b 100644
--- a/xmodule/annotator_mixin.py
+++ b/xmodule/annotator_mixin.py
@@ -37,6 +37,13 @@ def __init__(self):
         self.reset()
         self.fed = []
 
+    def handle_starttag(self, tag, attrs):
+        if tag != 'img':
+            return
+        for attr in attrs:
+            if len(attr) >= 2 and attr[0] == 'alt':
+                self.fed.append(attr[1])
+
     def handle_data(self, data):
         """takes the data in separate chunks"""
         self.fed.append(data)

From e6f33c6958d18bed9f78df7ac8fa9560edc83539 Mon Sep 17 00:00:00 2001
From: Navin Karkera <navin@opencraft.com>
Date: Thu, 6 Mar 2025 12:05:01 +0530
Subject: [PATCH 2/2] fixup! feat: preserve image alt text while converting
 html to text

---
 xmodule/tests/test_annotator_mixin.py | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/xmodule/tests/test_annotator_mixin.py b/xmodule/tests/test_annotator_mixin.py
index baaa7ad5d905..655d18ef343e 100644
--- a/xmodule/tests/test_annotator_mixin.py
+++ b/xmodule/tests/test_annotator_mixin.py
@@ -22,6 +22,10 @@ class HelperFunctionTest(unittest.TestCase):
     sample_sourceurl = "http://video-js.zencoder.com/oceans-clip.mp4"
     sample_youtubeurl = "http://www.youtube.com/watch?v=yxLIu-scR9Y"
     sample_html = '<p><b>Testing here</b> and not bolded here</p>'
+    # pylint: disable=line-too-long
+    sample_html_with_image_alt = '''<p>Testing here with image: </p><p><img src="/static/image.jpg" alt="the alt text" width="560" height="315" /></p>'''
+    # pylint: disable=line-too-long
+    sample_html_with_no_image_alt = '''<p>Testing here with image: </p><p><img src="/static/image.jpg" width="560" height="315" /></p>'''
 
     def test_get_instructions(self):
         """
@@ -54,3 +58,13 @@ def test_html_to_text(self):
         expectedtext = "Testing here and not bolded here"
         result = html_to_text(self.sample_html)
         assert expectedtext == result
+
+    def test_html_image_with_alt_text(self):
+        expectedtext = "Testing here with image: the alt text"
+        result = html_to_text(self.sample_html_with_image_alt)
+        assert expectedtext == result
+
+    def test_html_image_with_no_alt_text(self):
+        expectedtext = "Testing here with image: "
+        result = html_to_text(self.sample_html_with_no_image_alt)
+        assert expectedtext == result