From fca9c765aee2bc70da8338999eede8b531e47abd Mon Sep 17 00:00:00 2001
From: yomguy <yomguy@parisson.com>
Date: Thu, 10 Nov 2011 13:52:36 +0100
Subject: [PATCH] display related media as dl fields

---
 .../telemeta_default/mediaitem_detail.html    | 72 ++++++++++---------
 .../telemeta_default/mediaitem_detail_dc.html | 39 +++++-----
 2 files changed, 60 insertions(+), 51 deletions(-)

diff --git a/telemeta/templates/telemeta_default/mediaitem_detail.html b/telemeta/templates/telemeta_default/mediaitem_detail.html
index c2263c75..74ea26ec 100644
--- a/telemeta/templates/telemeta_default/mediaitem_detail.html
+++ b/telemeta/templates/telemeta_default/mediaitem_detail.html
@@ -277,61 +277,69 @@
         <div>
             <h4><a href="#">{% trans "Related media" %}</a></h4>
             {% if related_files %}
-            <div class="instruments">
+            <div>
                 <table class="instruments" width="100%">
                     <thead>
                         <tr>
-                            <td>{% field_label "MediaItemRelated" "title" %}</td>
-                            <td>{% field_label "MediaItemRelated" "description" %}</td>
-                            <td>{% field_label "MediaItemRelated" "mime type" %}</td>
+                            <td>{% field_label "MediaItemRelated" "media" %}</td>
                             <td>{% field_label "MediaItemRelated" "preview" %}</td>
                         </tr>
                     </thead>
                     <tbody>
                         {% for file in related_files %}
                         <tr>
-                            <td>
-                            {% if file.url %}
-                               <a href="{{ file.url }}" target="_blank">
-                                {% if file.title %}
-                                 {{ file.title }}
-                                {% else %} 
-                                 {{ file.url|get_filename }}
-                                {% endif %}
-                                </a>
-                            {% endif %}
-                            {% if file.file %}
-                              <a href="{% url telemeta-item-related-file item.public_id file.id %}" target="_blank">
-                                {% if file.title %}
-                                 {{ file.title }}
-                                {% else %} 
-                                 {{ file.file|get_filename }}
+                            <td style="font-size: 1em;">
+                            <dl class="listing">
+                                <dt>{% trans "Title" %}</dt>
+                                <dd>
+                                    {% if file.url %}
+                                        <a href="{{ file.url }}" target="_blank">
+                                        {% if file.title %}
+                                            {{ file.title }}</a>
+                                        {% else %}
+                                            {{ file.url|get_filename }}</a>
+                                        {% endif %}
+                                    {% else %}
+                                        <a href="{% url telemeta-item-related-file item.public_id file.id %}" target="_blank">
+                                        {% if file.title %}
+                                            {{ file.title }}</a>
+                                        {% else %}
+                                            {{ file.file|get_filename }}</a>
+                                        {% endif %} 
+                                    {% endif %}
+                                </dd>
+                                <dt>{% trans "Description" %}</dt>
+                                <dd>{{ file.description|html_line_break|safe }}</dd>
+                                {% dl_field file "mime_type" %}
+                                <dt>{% trans "URL" %}</dt>
+                                <dd>
+                                {% if file.url %}
+                                    <a href="{{ file.url }}" target="_blank">{{ file.url }}</a>
+                                {% else %}
+                                    <a href="{% url telemeta-item-related-file item.public_id file.id %}" target="_blank">
+                                        {% url telemeta-item-related-file item.public_id file.id %}
+                                    </a>
                                 {% endif %}
-                              </a>
-                            {% endif %}
+                                </dd>
+                            </dl>
                             </td>
-                            <td>{{ file.description }}</td>
-                            <td>{{ file.mime_type }}</td>
+
                             <td>{% if file.is_image %}
                                   {% if file.url %}
                                    <a href="{{ file.url }}">
-                                   <img src="{{ file.url }}" style="max-width: 420px;" /></a>
+                                   <img src="{{ file.url }}" style="max-width: 420px; max-height: 200px;" /></a>
                                   {% else %}
                                    <a href="{% url telemeta-item-related-file item.public_id file.id %}">
-                                   <img src="{% url telemeta-item-related-file item.public_id file.id %}" style="max-width: 420px;" /></a>
+                                   <img src="{% url telemeta-item-related-file item.public_id file.id %}" style="max-width: 420px; max-height: 200px;" /></a>
                                    {% endif %}
                                 {% else %}
                                 {% if file.url %}
                                  {% if "youtube" in file.url %}
-                                 <iframe width="420" height="315" src="{{ file.url|get_youtube }}" frameborder="0" allowfullscreen></iframe>
-                                 {% else %}
-                                 <a href="{{ file.url }}">{% trans "link" %}</a>
+                                 <iframe width="251" height="200" src="{{ file.url|get_youtube }}" frameborder="0" allowfullscreen></iframe>
                                  {% endif %}
                                 {% endif %}
-                                {% if file.file %}
-                                <a href="{% url telemeta-item-related-file item.public_id file.id %}">link</a>
                                 {% endif %}
-                                {% endif %}</td>
+                            </td>
                         </tr>
                         {% endfor %}
                     </tbody>
diff --git a/telemeta/templates/telemeta_default/mediaitem_detail_dc.html b/telemeta/templates/telemeta_default/mediaitem_detail_dc.html
index 0347be9c..d745dc67 100644
--- a/telemeta/templates/telemeta_default/mediaitem_detail_dc.html
+++ b/telemeta/templates/telemeta_default/mediaitem_detail_dc.html
@@ -1,19 +1,20 @@
-{% extends "telemeta/mediaitem_detail.html" %}
-{% load telemeta_utils %}
-{% load i18n %}
-
-{% if item %}
-{% block title_buttons %}
- <a class="component_icon button icon_previous" href="{% url telemeta-item-detail item.public_id %}">{% trans "Normal View" %}</a>
-{% endblock %}
-
-{% block infos %}
-{% with item|to_dublincore as resource %}
-{% include "telemeta/inc/dublincore.html" %}
-{% endwith %}
-{% endblock %}
-{% else %}
-    <p>{% trans "No such item" %}</p>
-{% endif %}
-
-
+{% extends "telemeta/mediaitem_detail.html" %}
+{% load telemeta_utils %}
+{% load i18n %}
+
+{% if item %}
+{% block title_buttons %}
+ <a class="component_icon button icon_previous" href="{% url telemeta-item-detail item.public_id %}">{% trans "Normal View" %}</a>
+{% endblock %}
+
+{% block infos %}
+{% with item|to_dublincore as resource %}
+{% include "telemeta/inc/dublincore.html" %}
+{% endwith %}
+{% endblock %}
+{% else %}
+    <p>{% trans "No such item" %}</p>
+{% endif %}
+
+{% block related %}
+{% endblock related %}
-- 
2.39.5