From ad879bc9f55fcf1b06f5ed7cf1a5fac7cc01d3b7 Mon Sep 17 00:00:00 2001
From: Pierre Jaury <pierre@jaury.eu>
Date: Tue, 22 Mar 2016 21:10:53 +0100
Subject: [PATCH] Add a confirmation dialog before deleting items

---
 admin/freeposte/admin/templates/alias/list.html  | 2 +-
 admin/freeposte/admin/templates/domain/list.html | 2 +-
 admin/freeposte/admin/templates/login.html       | 2 +-
 admin/freeposte/admin/templates/user/list.html   | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/admin/freeposte/admin/templates/alias/list.html b/admin/freeposte/admin/templates/alias/list.html
index 342b1558..c595466a 100644
--- a/admin/freeposte/admin/templates/alias/list.html
+++ b/admin/freeposte/admin/templates/alias/list.html
@@ -27,7 +27,7 @@ Alias list
     <tr>
       <td>
         <a href="{{ url_for('.alias_edit', alias=alias.get_id()) }}" title="Edit"><i class="fa fa-pencil"></i></a>&nbsp;
-        <a href="{{ url_for('.alias_delete', alias=alias.get_id()) }}" title="Delete"><i class="fa fa-trash"></i></a>
+        <a href="{{ url_for('.alias_delete', alias=alias.get_id()) }}" onclick="return confirm('Are you sure?')" title="Delete"><i class="fa fa-trash"></i></a>
       </td>
       <td>{{ alias }}</td>
       <td>{{ alias.destination or '-' }}</td>
diff --git a/admin/freeposte/admin/templates/domain/list.html b/admin/freeposte/admin/templates/domain/list.html
index d1b385b9..c02efd0d 100644
--- a/admin/freeposte/admin/templates/domain/list.html
+++ b/admin/freeposte/admin/templates/domain/list.html
@@ -28,8 +28,8 @@ Domain list
         <a href="{{ url_for('.user_list', domain_name=domain.name) }}" title="Users"><i class="fa fa-envelope-o"></i></a>&nbsp;
         <a href="{{ url_for('.alias_list', domain_name=domain.name) }}" title="Aliases"><i class="fa fa-at"></i></a>&nbsp;
         <a href="{{ url_for('.domain_admins', domain_name=domain.name) }}" title="Administrators"><i class="fa fa-user"></i></a>&nbsp;
-        <a href="{{ url_for('.domain_delete', domain_name=domain.name) }}" title="Delete"><i class="fa fa-trash"></i></a>&nbsp;
         <a href="{{ url_for('.domain_edit', domain_name=domain.name) }}" title="Edit"><i class="fa fa-pencil"></i></a>
+        <a href="{{ url_for('.domain_delete', domain_name=domain.name) }}" onclick="return confirm('Are you sure?')" title="Delete"><i class="fa fa-trash"></i></a>&nbsp;
       </td>
       <td>{{ domain.name }}</td>
       <td>{{ domain.users | count }} / {{ domain.max_users or '∞' }}</td>
diff --git a/admin/freeposte/admin/templates/login.html b/admin/freeposte/admin/templates/login.html
index a9f5f3de..3a217046 100644
--- a/admin/freeposte/admin/templates/login.html
+++ b/admin/freeposte/admin/templates/login.html
@@ -22,5 +22,5 @@ to access the administration tools
 {% endblock %}
 
 {% block box_content %}
-{{ wtf.quick_form(form) }}
+{{ macros.form(form) }}
 {% endblock %}
diff --git a/admin/freeposte/admin/templates/user/list.html b/admin/freeposte/admin/templates/user/list.html
index 835c80c8..2a14629d 100644
--- a/admin/freeposte/admin/templates/user/list.html
+++ b/admin/freeposte/admin/templates/user/list.html
@@ -33,7 +33,7 @@ User list
         <a href="{{ url_for('.user_reply', user_email=user.get_id()) }}" title="Auto-reply"><i class="fa fa-plane"></i></a>&nbsp;
         <a href="{{ url_for('.user_fetchmail', user_email=user.get_id()) }}" title="Fetched accounts"><i class="fa fa-download"></i></a>&nbsp;
         <a href="{{ url_for('.user_edit', user_email=user.get_id()) }}" title="Edit"><i class="fa fa-pencil"></i></a>&nbsp;
-        <a href="{{ url_for('.user_delete', user_email=user.get_id()) }}" title="Delete"><i class="fa fa-trash"></i></a>
+        <a href="{{ url_for('.user_delete', user_email=user.get_id()) }}" onclick="return confirm('Are you sure?')" title="Delete"><i class="fa fa-trash"></i></a>
       </td>
       <td>{{ user }}</td>
       <td>