From 766567ec4948f23d1dd6abb2c85ca6c1222a944a Mon Sep 17 00:00:00 2001
From: Roberto Rosario <Roberto.Rosario.Gonzalez@gmail.com>
Date: Sun, 1 May 2011 23:10:51 -0400
Subject: [PATCH] Added alt attribute to several img html tags

---
 apps/documents/__init__.py |  5 +++--
 apps/documents/forms.py    | 19 +++++++++++--------
 apps/ocr/views.py          |  5 +++--
 3 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/apps/documents/__init__.py b/apps/documents/__init__.py
index d24b0a06b4..4d86338c1d 100644
--- a/apps/documents/__init__.py
+++ b/apps/documents/__init__.py
@@ -127,10 +127,11 @@ def document_exists(document):
 
 register_model_list_columns(Document, [
         {'name':_(u'thumbnail'), 'attribute':
-            lambda x: u'<a class="fancybox" href="%(url)s"><img class="lazy-load" data-href="%(thumbnail)s" src="%(media_url)s/images/ajax-loader.gif" /><noscript><img src="%(thumbnail)s" /></noscript></a>' % {
+            lambda x: u'<a class="fancybox" href="%(url)s"><img class="lazy-load" data-href="%(thumbnail)s" src="%(media_url)s/images/ajax-loader.gif" alt="%(string)s" /><noscript><img src="%(thumbnail)s" alt="%(string)s" /></noscript></a>' % {
                 'url': reverse('document_preview', args=[x.pk]),
                 'thumbnail': reverse('document_thumbnail', args=[x.pk]),
-                'media_url': settings.MEDIA_URL
+                'media_url': settings.MEDIA_URL,
+                'string': _(u'thumbnail')
             }
         },
         {'name':_(u'metadata'), 'attribute':
diff --git a/apps/documents/forms.py b/apps/documents/forms.py
index ac9d8f8aa6..aba4579f16 100644
--- a/apps/documents/forms.py
+++ b/apps/documents/forms.py
@@ -40,9 +40,9 @@ class DocumentPageImageWidget(forms.widgets.Widget):
             output.append('''
                 <div class="full-height scrollable" style="overflow: auto;">
                     <div class="tc">
-                        <img class="lazy-load" data-href="%(img)s?page=%(page)d&zoom=%(zoom)d&rotation=%(rotation)d" src="%(media_url)s/images/ajax-loader.gif" />
+                        <img class="lazy-load" data-href="%(img)s?page=%(page)d&zoom=%(zoom)d&rotation=%(rotation)d" src="%(media_url)s/images/ajax-loader.gif" alt="%(string)s" />
                         <noscript>
-                            <img src="%(img)s?page=%(page)d&zoom=%(zoom)d&rotation=%(rotation)d" />
+                            <img src="%(img)s?page=%(page)d&zoom=%(zoom)d&rotation=%(rotation)d" alt="%(string)s" />
                         </noscript>     
                     </div>    
                 </div>''' % {
@@ -50,7 +50,8 @@ class DocumentPageImageWidget(forms.widgets.Widget):
                 'page': value.page_number,
                 'zoom': zoom,
                 'rotation': rotation,
-                'media_url': settings.MEDIA_URL
+                'media_url': settings.MEDIA_URL,
+                'string': ugettext(u'page image')
                 })
             return mark_safe(u''.join(output))
         else:
@@ -118,9 +119,9 @@ class ImageWidget(forms.widgets.Widget):
                         <div class="tc">%(page_string)s %(page)s</div>
                         <div class="tc">
                             <a rel="page_gallery" class="fancybox-noscaling" href="%(view_url)s?page=%(page)d">
-                                <img class="lazy-load" data-href="%(img)s?page=%(page)d" src="%(media_url)s/images/ajax-loader.gif"/>
+                                <img class="lazy-load" data-href="%(img)s?page=%(page)d" src="%(media_url)s/images/ajax-loader.gif" alt="%(string)s" />
                                 <noscript>
-                                    <img src="%(img)s?page=%(page)d" />
+                                    <img src="%(img)s?page=%(page)d" alt="%(string)s" />
                                 </noscript>
                             </a>
                         </div>
@@ -135,6 +136,7 @@ class ImageWidget(forms.widgets.Widget):
                     'page_string': ugettext(u'Page'),
                     'details_string': ugettext(u'Details'),
                     'media_url': settings.MEDIA_URL,
+                    'string': _(u'document page')
                 })
 
         output.append(u'</div>')
@@ -384,9 +386,9 @@ class MetaDataImageWidget(forms.widgets.Widget):
                         %(tags_template)s
                         <div class="tc">
                             <a rel="group_%(group_id)d_documents_gallery" class="fancybox-noscaling" href="%(view_url)s">
-                                <img class="lazy-load" style="border: 1px solid black; margin: 10px;" src="%(media_url)s/images/ajax-loader.gif" data-href="%(img)s" />
+                                <img class="lazy-load" style="border: 1px solid black; margin: 10px;" src="%(media_url)s/images/ajax-loader.gif" data-href="%(img)s" alt="%(string)s" />
                                 <noscript>
-                                    <img style="border: 1px solid black; margin: 10px;" src="%(img)s" />
+                                    <img style="border: 1px solid black; margin: 10px;" src="%(img)s" alt="%(string)s" />
                                 </noscript>
                             </a>
                         </div>
@@ -404,7 +406,8 @@ class MetaDataImageWidget(forms.widgets.Widget):
                     'group_id': value['group'].pk,
                     'document_name': document,
                     'media_url': settings.MEDIA_URL,
-                    'tags_template': u''.join(tags_template) if tags_template else u''
+                    'tags_template': u''.join(tags_template) if tags_template else u'',
+                    'string': _(u'group document'),
                 })
         output.append(u'</div>')
         output.append(
diff --git a/apps/ocr/views.py b/apps/ocr/views.py
index 162d66c1da..d203f46f62 100644
--- a/apps/ocr/views.py
+++ b/apps/ocr/views.py
@@ -27,10 +27,11 @@ from ocr.api import clean_pages
 
 def _display_thumbnail(ocr_document):
     try:
-        return u'<a class="fancybox" href="%(url)s"><img class="lazy-load" data-href="%(thumbnail_url)s" src="%(media_url)s/images/ajax-loader.gif" /><noscript><img src="%(thumbnail_url)s" /></noscript></a>' % {
+        return u'<a class="fancybox" href="%(url)s"><img class="lazy-load" data-href="%(thumbnail_url)s" src="%(media_url)s/images/ajax-loader.gif" alt="%(string)s" /><noscript><img src="%(thumbnail_url)s" alt="%(string)s" /></noscript></a>' % {
             'url': reverse('document_preview', args=[ocr_document.document.pk]),
             'thumbnail_url': reverse('document_thumbnail', args=[ocr_document.document.pk]),
-            'media_url': settings.MEDIA_URL
+            'media_url': settings.MEDIA_URL,
+            'string': _(u'thumbnail')
         }
     except:
         return u''