From 0fc8b0ee4977ef00f004e3990ce8621ac63dec1d Mon Sep 17 00:00:00 2001
From: Herculino Trotta <netotrotta@gmail.com>
Date: Sun, 5 Jan 2025 01:35:25 -0300
Subject: [PATCH] feat(tags): add tab to show archived tags

---
 app/apps/transactions/urls.py           |  2 +
 app/apps/transactions/views/tags.py     | 24 +++++++++++
 app/templates/tags/fragments/list.html  | 55 ++++++-------------------
 app/templates/tags/fragments/table.html | 55 +++++++++++++++++++++++++
 app/templates/tags/pages/index.html     |  2 +-
 5 files changed, 94 insertions(+), 44 deletions(-)
 create mode 100644 app/templates/tags/fragments/table.html

diff --git a/app/apps/transactions/urls.py b/app/apps/transactions/urls.py
index 09ec2a5..6c1c4b8 100644
--- a/app/apps/transactions/urls.py
+++ b/app/apps/transactions/urls.py
@@ -53,6 +53,8 @@ urlpatterns = [
     ),
     path("tags/", views.tags_index, name="tags_index"),
     path("tags/list/", views.tags_list, name="tags_list"),
+    path("tags/table/active/", views.tags_table_active, name="tags_table_active"),
+    path("tags/table/archived/", views.tags_table_archived, name="tags_table_archived"),
     path("tags/add/", views.tag_add, name="tag_add"),
     path(
         "tags/<int:tag_id>/edit/",
diff --git a/app/apps/transactions/views/tags.py b/app/apps/transactions/views/tags.py
index 9ba06ba..354208b 100644
--- a/app/apps/transactions/views/tags.py
+++ b/app/apps/transactions/views/tags.py
@@ -32,6 +32,30 @@ def tags_list(request):
     )
 
 
+@only_htmx
+@login_required
+@require_http_methods(["GET"])
+def tags_table_active(request):
+    tags = TransactionTag.objects.filter(active=True).order_by("id")
+    return render(
+        request,
+        "tags/fragments/table.html",
+        {"tags": tags, "active": True},
+    )
+
+
+@only_htmx
+@login_required
+@require_http_methods(["GET"])
+def tags_table_archived(request):
+    tags = TransactionTag.objects.filter(active=False).order_by("id")
+    return render(
+        request,
+        "tags/fragments/table.html",
+        {"tags": tags, "active": False},
+    )
+
+
 @only_htmx
 @login_required
 @require_http_methods(["GET", "POST"])
diff --git a/app/templates/tags/fragments/list.html b/app/templates/tags/fragments/list.html
index 61355e9..82850ed 100644
--- a/app/templates/tags/fragments/list.html
+++ b/app/templates/tags/fragments/list.html
@@ -15,49 +15,18 @@
   </div>
 
   <div class="card">
-    <div class="card-body table-responsive">
-      {% if tags %}
-      <c-config.search></c-config.search>
-      <table class="table table-hover">
-        <thead>
-          <tr>
-            <th scope="col" class="col-auto"></th>
-            <th scope="col" class="col">{% translate 'Name' %}</th>
-          </tr>
-        </thead>
-        <tbody>
-          {% for tag in tags %}
-          <tr class="tag">
-            <td class="col-auto">
-              <div class="btn-group" role="group" aria-label="{% translate 'Actions' %}">
-                <a class="btn btn-secondary btn-sm"
-                      role="button"
-                      data-bs-toggle="tooltip"
-                      data-bs-title="{% translate "Edit" %}"
-                      hx-get="{% url 'tag_edit' tag_id=tag.id %}"
-                      hx-target="#generic-offcanvas">
-                  <i class="fa-solid fa-pencil fa-fw"></i></a>
-                <a class="btn btn-secondary btn-sm text-danger"
-                        role="button"
-                        data-bs-toggle="tooltip"
-                        data-bs-title="{% translate "Delete" %}"
-                        hx-delete="{% url 'tag_delete' tag_id=tag.id %}"
-                        hx-trigger='confirmed'
-                        data-bypass-on-ctrl="true"
-                        data-title="{% translate "Are you sure?" %}"
-                        data-text="{% translate "You won't be able to revert this!" %}"
-                        data-confirm-text="{% translate "Yes, delete it!" %}"
-                        _="install prompt_swal"><i class="fa-solid fa-trash fa-fw"></i></a>
-              </div>
-            </td>
-            <td class="col">{{ tag.name }}</td>
-          </tr>
-          {% endfor %}
-        </tbody>
-      </table>
-      {% else %}
-      <c-msg.empty title="{% translate "No tags" %}" remove-padding></c-msg.empty>
-      {% endif %}
+    <div class="card-header">
+      <ul class="nav nav-pills card-header-pills" id="myTab" role="tablist">
+        <li class="nav-item" role="presentation">
+          <button class="nav-link active" data-bs-toggle="tab" type="button" role="tab" aria-selected="true" hx-get="{% url 'tags_table_active' %}" hx-trigger="load, click" hx-target="#tags-table">{% translate 'Active' %}</button>
+        </li>
+        <li class="nav-item" role="presentation">
+          <button class="nav-link" hx-get="{% url 'tags_table_archived' %}" hx-target="#tags-table" data-bs-toggle="tab" type="button" role="tab" aria-selected="false">{% translate 'Archived' %}</button>
+        </li>
+      </ul>
+    </div>
+    <div class="card-body">
+      <div id="tags-table"></div>
     </div>
   </div>
 </div>
diff --git a/app/templates/tags/fragments/table.html b/app/templates/tags/fragments/table.html
new file mode 100644
index 0000000..cc874ee
--- /dev/null
+++ b/app/templates/tags/fragments/table.html
@@ -0,0 +1,55 @@
+{% load i18n %}
+{% if active %}
+  <div class="show-loading" hx-get="{% url 'tags_table_active' %}" hx-trigger="updated from:window"
+       hx-swap="outerHTML">
+{% else %}
+  <div class="show-loading" hx-get="{% url 'tags_table_archived' %}" hx-trigger="updated from:window"
+       hx-swap="outerHTML">
+{% endif %}
+  <div class="table-responsive">
+    {% if tags %}
+      <c-config.search></c-config.search>
+      <table class="table table-hover">
+        <thead>
+        <tr>
+          <th scope="col" class="col-auto"></th>
+          <th scope="col" class="col">{% translate 'Name' %}</th>
+        </tr>
+        </thead>
+        <tbody>
+        {% for tag in tags %}
+          <tr class="tag">
+            <td class="col-auto">
+              <div class="btn-group" role="group" aria-label="{% translate 'Actions' %}">
+                <a class="btn btn-secondary btn-sm"
+                   role="button"
+                   hx-swap="innerHTML"
+                   data-bs-toggle="tooltip"
+                   data-bs-title="{% translate "Edit" %}"
+                   hx-get="{% url 'tag_edit' tag_id=tag.id %}"
+                   hx-target="#generic-offcanvas">
+                  <i class="fa-solid fa-pencil fa-fw"></i></a>
+                <a class="btn btn-secondary btn-sm text-danger"
+                   role="button"
+                   data-bs-toggle="tooltip"
+                   hx-swap="innerHTML"
+                   data-bs-title="{% translate "Delete" %}"
+                   hx-delete="{% url 'tag_delete' tag_id=tag.id %}"
+                   hx-trigger='confirmed'
+                   data-bypass-on-ctrl="true"
+                   data-title="{% translate "Are you sure?" %}"
+                   data-text="{% translate "You won't be able to revert this!" %}"
+                   data-confirm-text="{% translate "Yes, delete it!" %}"
+                   _="install prompt_swal"><i class="fa-solid fa-trash fa-fw"></i></a>
+              </div>
+            </td>
+            <td class="col">{{ tag.name }}</td>
+          </tr>
+        {% endfor %}
+        </tbody>
+      </table>
+    {% else %}
+      <c-msg.empty title="{% translate "No tags" %}" remove-padding></c-msg.empty>
+    {% endif %}
+  </div>
+</div>
diff --git a/app/templates/tags/pages/index.html b/app/templates/tags/pages/index.html
index a8fce6b..1729636 100644
--- a/app/templates/tags/pages/index.html
+++ b/app/templates/tags/pages/index.html
@@ -4,5 +4,5 @@
 {% block title %}{% translate 'Tags' %}{% endblock %}
 
 {% block content %}
-<div hx-get="{% url 'tags_list' %}" hx-trigger="load, updated from:window" class="show-loading"></div>
+<div hx-get="{% url 'tags_list' %}" hx-trigger="load" class="show-loading"></div>
 {% endblock %}