From c31036a29b04f7e6c8e8e87e8c57653f380d5cc1 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Stefan=20K=C3=B6gl?= <stefan@skoegl.net>
Date: Sat, 15 Sep 2012 18:24:46 +0200
Subject: [PATCH] fix display issues on podcast page

---
 mygpo/web/templates/podcast.html | 14 ++++++++++----
 mygpo/web/views/podcast.py       |  5 +----
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/mygpo/web/templates/podcast.html b/mygpo/web/templates/podcast.html
index 73473c3f..4a82dd61 100755
--- a/mygpo/web/templates/podcast.html
+++ b/mygpo/web/templates/podcast.html
@@ -147,7 +147,7 @@
         </td>
         <td style="text-align: center;">
          <form class="form-inline" method="post" action="{% podcast_link_target podcast "unsubscribe" device.uid %}?return_to={% podcast_link_target podcast %}">
-          <button class="btn btn-danger" type="submit">
+          <button class="btn btn-danger btn-small" type="submit">
            <i class="icon-remove"></i>
           </button>
          </form>
@@ -157,9 +157,15 @@
       {% if can_subscribe %}
        <tr>
         <form class="form-inline" action="{% podcast_link_target podcast "subscribe" %}" method="post">
-         {% csrf_token %}
-         <td>{{ subscribe_form.as_p }}</td>
-         <td style="text-align: center;">
+         <td>
+          {% csrf_token %}
+          <select name="targets" id="id_targets">
+           {% for device in subscribe_targets %}
+            <option value="{{ device.uid }}">{{ device.name }}</option>
+           {% endfor %}
+          </select>
+         </td>
+         <td>
           <button class="btn btn-success btn-small" type="submit">
            <i class="icon-ok"></i>
           </button>
diff --git a/mygpo/web/views/podcast.py b/mygpo/web/views/podcast.py
index 06f24b8b..9f334011 100644
--- a/mygpo/web/views/podcast.py
+++ b/mygpo/web/views/podcast.py
@@ -88,9 +88,6 @@ def show(request, podcast):
 
         is_public = state.settings.get('public_subscription', True)
 
-        subscribe_form = SyncForm()
-        subscribe_form.set_targets(subscribe_targets, '')
-
         return render(request, 'podcast.html', {
             'tags': tags,
             'history': history,
@@ -99,7 +96,7 @@ def show(request, podcast):
             'devices': subscribed_devices,
             'related_podcasts': rel_podcasts,
             'can_subscribe': len(subscribe_targets) > 0,
-            'subscribe_form': subscribe_form,
+            'subscribe_targets': subscribe_targets,
             'episode': episode,
             'episodes': episodes,
             'max_listeners': max_listeners,
-- 
2.11.4.GIT