From 229843cd1eb961ca6975d787aa5e68780e54afac Mon Sep 17 00:00:00 2001
From: Sunil Mohan Adapa <sunil@medhas.org>
Date: Mon, 29 Jul 2019 14:04:18 -0700
Subject: [PATCH] backups: Fix HTML template indentation, remove inline styling

Remove inline styling as it won't work with upcoming content security rules.
They should be re-added in CSS files.

Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
---
 .../backups/templates/backups_repository.inc  | 65 +++++++++----------
 1 file changed, 31 insertions(+), 34 deletions(-)

diff --git a/plinth/modules/backups/templates/backups_repository.inc b/plinth/modules/backups/templates/backups_repository.inc
index 94a733b52..57401c952 100644
--- a/plinth/modules/backups/templates/backups_repository.inc
+++ b/plinth/modules/backups/templates/backups_repository.inc
@@ -24,8 +24,7 @@
   <thead class="collapsible-button" data-toggle="collapse" data-target="#{{ uuid }}">
     <tr>
       <th colspan="2">
-        <span style="vertical-align: text-top">
-          <span class="fa fa-chevron-right fa-fw" aria-hidden="true"></span>
+        <span class="fa fa-chevron-right fa-fw" aria-hidden="true"></span>
         {% if repository.error %}
           <span class="fa fa-exclamation-triangle mount-error"
                 aria-hidden="true" title="{{ repository.error }}">
@@ -35,44 +34,42 @@
         {{ repository.name }}
 
         <span class="pull-right">
-        {% if editable %}
+          {% if editable %}
 
-          {% if repository.mounted %}
+            {% if repository.mounted %}
 
-            <!-- With GET redirects, the browser URL points to the
-                 redirected page (bad when reloading) - use POST instead.
-            -->
-            <form action="{% url 'backups:repository-umount' uuid %}" method="POST"
-                  class="inline-block" >
-              {% csrf_token %}
-              <button type="submit" class="btn btn-sm btn-default"
-                     title="{% trans 'Unmount Location' %}">
-               <span class="fa fa-eject" aria-hidden="true">
-               </button>
-            </form>
+              <!-- With GET redirects, the browser URL points to the
+                   redirected page (bad when reloading) - use POST instead.
+              -->
+              <form action="{% url 'backups:repository-umount' uuid %}" method="POST"
+                    class="inline-block" >
+                {% csrf_token %}
+                <button type="submit" class="btn btn-sm btn-default"
+                       title="{% trans 'Unmount Location' %}">
+                  <span class="fa fa-eject" aria-hidden="true"></span>
+                </button>
+              </form>
 
-          {% else %}
+            {% else %}
 
-            <form action="{% url 'backups:repository-mount' uuid %}" method="POST"
-                  class="inline-block" >
-              {% csrf_token %}
-              <button type="submit" class="btn btn-sm btn-default"
-                     title="{% trans 'Mount Location' %}">
-               <span class="fa fa-eye" aria-hidden="true">
-               </button>
-            </form>
+              <form action="{% url 'backups:repository-mount' uuid %}" method="POST"
+                    class="inline-block" >
+                {% csrf_token %}
+                <button type="submit" class="btn btn-sm btn-default"
+                        title="{% trans 'Mount Location' %}">
+                  <span class="fa fa-eye" aria-hidden="true"></span>
+                </button>
+              </form>
+
+            {% endif %}
+
+            <a title="{% trans 'Remove Backup Location. This will not delete the remote backup.' %}"
+               role="button" class="repository-remove btn btn-sm btn-default"
+               href="{% url 'backups:repository-remove' uuid %}">
+              <span class="fa fa-trash-o" aria-hidden="true"></span>
+            </a>
 
           {% endif %}
-
-          <a title="{% trans 'Remove Backup Location. This will not delete the remote backup.' %}"
-             role="button" class="repository-remove btn btn-sm btn-default"
-             href="{% url 'backups:repository-remove' uuid %}">
-            <span class="fa fa-trash-o" aria-hidden="true">
-          </a>
-
-        {% endif %}
-        </span>
-
         </span>
       </th>
     </tr>