Use real image stub sizes, not 200x150

boards/models/attachment/viewers.py

@@ -1,6 +1,9 b''
+import os
+
 from django.core.files.images import get_image_dimensions
 from django.template.defaultfilters import filesizeformat
 from django.contrib.staticfiles.templatetags.staticfiles import static
+from django.contrib.staticfiles import finders
 
 FILE_STUB_IMAGE = 'images/file.png'
 FILE_STUB_URL = 'url'
@@ -45,6 +48,11 b' def get_viewers():'
     return AbstractViewer.__subclasses__()
 
 
+def get_static_dimensions(filename):
+    file_path = finders.find(filename)
+    return get_image_dimensions(file_path)
+
+
 class AbstractViewer:
     def __init__(self, file, file_type, hash, url):
         self.file = file
@@ -70,9 +78,11 b' class AbstractViewer:'
         else:
             image = FILE_STUB_IMAGE
 
+        w, h = get_static_dimensions(image)
+
         return '<a href="{}">'\
-               '<img class="url-image" src="{}" width="200" height="150"/>'\
-               '</a>'.format(self.file.url, static(image))
+               '<img class="url-image" src="{}" width="{}" height="{}"/>'\
+               '</a>'.format(self.file.url, static(image), w, h)
 
 
 class VideoViewer(AbstractViewer):
@@ -153,6 +163,8 b' class UrlViewer(AbstractViewer):'
         url_image_name = URL_PROTOCOLS.get(protocol, FILE_STUB_URL)
         image = static('images/' + url_image_name + '.png')
 
+        w, h = get_static_dimensions('images/' + url_image_name + '.png')
+
         return '<a href="{}">' \
-               '<img class="url-image" src="{}" width="200" height="150"/>' \
-               '</a>'.format(self.url, image)
+               '<img class="url-image" src="{}" width="{}" height="{}"/>' \
+               '</a>'.format(self.url, image, w, h)