From d1e41848641672b0102ba85dc32285724a83fa94 Mon Sep 17 00:00:00 2001
From: Oliver Falk <oliver@linux-kernel.at>
Date: Wed, 4 Jul 2018 14:30:59 +0200
Subject: [PATCH 1/3] Link to the raw image on profile page

---
 ivatar/ivataraccount/templates/profile.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ivatar/ivataraccount/templates/profile.html b/ivatar/ivataraccount/templates/profile.html
index f5fe074..7878067 100644
--- a/ivatar/ivataraccount/templates/profile.html
+++ b/ivatar/ivataraccount/templates/profile.html
@@ -72,7 +72,7 @@
 <p>{% trans 'Here are the photos you have uploaded/imported:' %}</p>
 <ul class="horizontal-list avatar-list centered">
 {% for photo in user.photo_set.all %}
-<li><img class="thumbnail" src="{% url 'raw_image' photo.id %}" title="{% blocktrans with photo.add_date as datetime %}Uploaded on {{ datetime }}{% endblocktrans %}" alt="{% blocktrans with photo.add_date as datetime %}Uploaded on {{ datetime }}{% endblocktrans %}"><br>
+<li><a href="{% url 'raw_image' photo.id %}"><img class="thumbnail" src="{% url 'raw_image' photo.id %}" title="{% blocktrans with photo.add_date as datetime %}Uploaded on {{ datetime }}{% endblocktrans %}" alt="{% blocktrans with photo.add_date as datetime %}Uploaded on {{ datetime }}{% endblocktrans %}"></a><br>
     <a href="{% url 'delete_photo' photo.id %}">{% trans 'Delete' %}</a></li>
 {% endfor %}
 </ul>

From e599c52b00acc047401ee09209f680fc6a569b2a Mon Sep 17 00:00:00 2001
From: Oliver Falk <oliver@linux-kernel.at>
Date: Thu, 5 Jul 2018 12:47:49 +0200
Subject: [PATCH 2/3] Fix path to delete icon/image

---
 ivatar/ivataraccount/templates/profile.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ivatar/ivataraccount/templates/profile.html b/ivatar/ivataraccount/templates/profile.html
index 7878067..ad2d4f5 100644
--- a/ivatar/ivataraccount/templates/profile.html
+++ b/ivatar/ivataraccount/templates/profile.html
@@ -55,7 +55,7 @@
     {% endfor %}
     {% for openid in user.unconfirmedopenid_set.all %}
         <li><form action="{% url 'remove_unconfirmed_openid' openid.id %}" method="post">{% csrf_token %}{{ openid.openid }}
-        <input type="image" src="{% static '/static/img/delete_button.png' %}" value="{% trans 'Remove' %}"></form>{# TODO: (expires in xx hours) #}</li>
+        <input type="image" src="{% static '/img/delete_button.png' %}" value="{% trans 'Remove' %}"></form>{# TODO: (expires in xx hours) #}</li>
     {% endfor %}
 </ul>
 {% endif %}

From 469ff6dbab25a5cd1ce3bddd827b11b830a7eb86 Mon Sep 17 00:00:00 2001
From: Oliver Falk <oliver@linux-kernel.at>
Date: Thu, 5 Jul 2018 12:48:20 +0200
Subject: [PATCH 3/3] Do not add message about successful adding (unconfirmed
 OpenID)

---
 ivatar/ivataraccount/views.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ivatar/ivataraccount/views.py b/ivatar/ivataraccount/views.py
index 96a48ca..93ab528 100644
--- a/ivatar/ivataraccount/views.py
+++ b/ivatar/ivataraccount/views.py
@@ -374,7 +374,9 @@ class AddOpenIDView(SuccessMessageMixin, FormView):
         if not openid_id:
             return render(self.request, self.template_name, {'form': form})
         else:
-            messages.success(self.request, _('ID added successfully'))
+            # At this point we have an unconfirmed OpenID, but
+            # we do not add the message, that we successfully added it,
+            # since this is misleading
             return HttpResponseRedirect(
                 reverse_lazy('openid_redirection', args=[openid_id]))