From faac8cdd9c2a2367e0af3335213dadc0faabb4d9 Mon Sep 17 00:00:00 2001
From: Joseph Nuthalapati <njoseph@thoughtworks.com>
Date: Fri, 26 Jul 2019 17:30:21 +0530
Subject: [PATCH] backups: Make backup location tables collapsible

Signed-off-by: Joseph Nuthalapati <njoseph@thoughtworks.com>
[sunil@medhas.org Fix incorrectly removed <tr> tag in <thead>]
Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
Reviewed-by: Sunil Mohan Adapa <sunil@medhas.org>
---
 plinth/modules/backups/templates/backups_repository.inc | 7 ++++---
 static/themes/default/css/plinth.css                    | 6 +++---
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/plinth/modules/backups/templates/backups_repository.inc b/plinth/modules/backups/templates/backups_repository.inc
index 6b4752884..94a733b52 100644
--- a/plinth/modules/backups/templates/backups_repository.inc
+++ b/plinth/modules/backups/templates/backups_repository.inc
@@ -21,10 +21,11 @@
 
 <table class="table table-bordered table-condensed table-striped"
        id="archives-list">
-  <thead>
+  <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>
         {% if repository.error %}
           <span class="fa fa-exclamation-triangle mount-error"
                 aria-hidden="true" title="{{ repository.error }}">
@@ -63,7 +64,7 @@
 
           {% endif %}
 
-          <a title="{% trans 'Remove Location. This will not delete the remote backup.' %}"
+          <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">
@@ -77,7 +78,7 @@
     </tr>
   </thead>
 
-  <tbody>
+  <tbody class="collapse in" id="{{ uuid }}">
     {% if repository.mounted %}
 
       {% for archive in repository.archives %}
diff --git a/static/themes/default/css/plinth.css b/static/themes/default/css/plinth.css
index d0a9106f5..7f00054cf 100644
--- a/static/themes/default/css/plinth.css
+++ b/static/themes/default/css/plinth.css
@@ -235,12 +235,12 @@ footer {
     margin-left: 5px;
 }
 
-.collapsible-button .fa:before,
-.no-js .collapsible-button.collapsed .fa:before {
+.collapsible-button .fa-chevron-right:before,
+.no-js .collapsible-button.collapsed .fa-chevron-right:before {
     content: "\f078";
 }
 
-.collapsible-button.collapsed .fa:before {
+.collapsible-button.collapsed .fa-chevron-right:before {
     content: "\f054";
 }