From 44004c17e341019cf05050692a7058f72e377b45 Mon Sep 17 00:00:00 2001
From: Jef van Schendel <mail@jefvanschendel.nl>
Date: Fri, 8 Feb 2013 22:54:16 +0100
Subject: [PATCH] Fix "Add to collection" button: remove icon, add text, add to
 collections.html

---
 mediagoblin/static/images/icon_collect.png    | Bin 588 -> 0 bytes
 .../mediagoblin/user_pages/media.html         |  17 +------
 .../mediagoblin/utils/collections.html        |  43 ++++++++++++------
 3 files changed, 29 insertions(+), 31 deletions(-)
 delete mode 100644 mediagoblin/static/images/icon_collect.png

diff --git a/mediagoblin/static/images/icon_collect.png b/mediagoblin/static/images/icon_collect.png
deleted file mode 100644
index 2911af2401d64d623150234805b8ecdedd810e97..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 588
zcmV-S0<-;zP)<h;3K|Lk000e1NJLTq0015U0015c1ONa4kvV2500001b5ch_0Itp)
z=>Px#0%A)?L;wJ)jUzGu000SaNLh0L01FcU01FcV0GgZ_00007bV*G`2iyn|02UkP
z0CDsH00GcRL_t(I%jK0jXcSQthW}L%EF{|5N36`)&34-QAc_bU31Xv#MAS+xgh3G8
zLK56U><kGQNeI#SKo;Dfh%;`Hodh*tnjpl8Vpc^_W+vye$Qo9~&D~D$@6O?VoX0)q
zUg`hSTGxZ?@>`C6_^$cZLv-{5kT4!<XN&AO2{T+_frN6{5_&quO30HoY+u00<(<)X
z?=zSeEL#_W<AF<y{4q|Iuaj~u`spiywHF-VM^VZ{;5qV_Jv}jY2M6>(fX(p+tSi`;
zlcY9p5p0`7(tZ`ALP9T+{G04AUnONQa)?_DGpeM#L!NX$N+|&|D+M&s9xve|ufUGP
z((VA8XAbOyOP_C%Q6XalN&Zue3MoaGRvnA6S>BV-=}DKuB_>H(q|IBM^=cST$S8)=
zLGNq5fS%YPZZ8R!foYVNhUx{}bxFB9z-HMvfsG*@n!;Z2sbM^uY69$RAU!6tNw6iV
zzl>+IB`9F`kfdE5qkLhS^VT??O?5JWfyRB^`Wme3uPLwj{Y_fmU=I&$R$uPM1-IqR
zr@Te=``j^w)mAJAjf5#L89j3@xE*0wXWYu0<M)9XVANO*&Hd@xTj>9`NkcmPK+pcw
aAL9p&ia{)DnPl7m0000<MNUMnLSTYz1pKD}

diff --git a/mediagoblin/templates/mediagoblin/user_pages/media.html b/mediagoblin/templates/mediagoblin/user_pages/media.html
index 1cc2a763..7550c6c1 100644
--- a/mediagoblin/templates/mediagoblin/user_pages/media.html
+++ b/mediagoblin/templates/mediagoblin/user_pages/media.html
@@ -149,9 +149,7 @@
       {% include "mediagoblin/utils/tags.html" %}
     {% endif %}
 
-    {% if media.collections %}
-      {% include "mediagoblin/utils/collections.html" %}
-    {% endif %}
+    {% include "mediagoblin/utils/collections.html" %}
 
     {% include "mediagoblin/utils/license.html" %}
 
@@ -183,19 +181,6 @@
       </p>
     {% endif %}
 
-    {% if request.user %}
-      <p>
-        <a type="submit" href="{{ request.urlgen('mediagoblin.user_pages.media_collect', 
-                                    user=media.get_uploader.username,
-                                    media=media.id) }}"
-           class="button_action"
-           title="{% trans %}Add media to collection{% endtrans %}">
-          <img src="{{ request.staticdirect('/images/icon_collect.png') }}"
-               />
-	</a>
-      </p>
-    {% endif %}
-
     {% template_hook("media_sideinfo") %}
 
     {% block mediagoblin_sidebar %}
diff --git a/mediagoblin/templates/mediagoblin/utils/collections.html b/mediagoblin/templates/mediagoblin/utils/collections.html
index 0afd9ed8..d9e4c8e4 100644
--- a/mediagoblin/templates/mediagoblin/utils/collections.html
+++ b/mediagoblin/templates/mediagoblin/utils/collections.html
@@ -17,20 +17,33 @@
 #}
 
 {% block collections_content -%}
-  <h3>{% trans collected=media.collected %}Collected in{% endtrans %}</h3>
-  <p>
-    {% for collection in media.collections %}
-      {% if loop.last %}
-        {# the 'and' should only appear if there is more than one collections #}
-        {% if media.collections|length > 1 %}
-          &middot;
+  {% if media.collections %}
+    <h3>{% trans collected=media.collected %}Collected in{% endtrans %}</h3>
+    <p>
+      {% for collection in media.collections %}
+        {% if loop.last %}
+          {# the 'and' should only appear if there is more than one collections #}
+          {% if media.collections|length > 1 %}
+            &middot;
+          {% endif %}
+          <a href="{{ collection.url_for_self(request.urlgen) }}">{{ collection['title'] }}</a>
+        {% elif loop.revindex == 2 %}
+          <a href="{{ collection.url_for_self(request.urlgen) }}">{{ collection['title'] }}</a>
+        {% else %}
+          <a href="{{ collection.url_for_self(request.urlgen) }}">{{ collection['title'] }}</a> &middot;
         {% endif %}
-        <a href="{{ collection.url_for_self(request.urlgen) }}">{{ collection['title'] }}</a>
-      {% elif loop.revindex == 2 %}
-        <a href="{{ collection.url_for_self(request.urlgen) }}">{{ collection['title'] }}</a>
-      {% else %}
-        <a href="{{ collection.url_for_self(request.urlgen) }}">{{ collection['title'] }}</a> &middot;
-      {% endif %}
-    {% endfor %}
-  </p>
+      {% endfor %}
+    </p>
+  {% endif %}
+  {% if request.user %}
+    <p>
+      <a type="submit" href="{{ request.urlgen('mediagoblin.user_pages.media_collect', 
+                                  user=media.get_uploader.username,
+                                  media=media.id) }}"
+         class="button_action"
+         title="{% trans %}Add media to collection{% endtrans %}">
+        Add to a collection
+      </a>
+    </p>
+  {% endif %}
 {% endblock %}
-- 
2.25.1