From 21ef696057f51f96d9ac785031217ae81904d373 Mon Sep 17 00:00:00 2001
From: Thomas Rusiecki <rusieckithomas@gmail.com>
Date: Fri, 14 Feb 2025 17:51:11 -0300
Subject: [PATCH] minor view changes

---
 admin/templates/states_panel.html           |  8 ++----
 dashboard/templates/base.html               |  5 ++--
 dashboard/templates/unassigned_devices.html | 31 +++++++++++----------
 device/templates/details.html               |  5 ++--
 evidence/templates/ev_details.html          | 14 +++++-----
 evidence/templates/upload.html              |  4 +--
 user/templates/panel.html                   |  2 +-
 7 files changed, 33 insertions(+), 36 deletions(-)

diff --git a/admin/templates/states_panel.html b/admin/templates/states_panel.html
index 540cc42..78a21d9 100644
--- a/admin/templates/states_panel.html
+++ b/admin/templates/states_panel.html
@@ -9,13 +9,9 @@
 {% endblock %}
 
 {% block content %}
-<div class="row">
-  <div class="col">
-    <h3>{{ subtitle }}</h3>
-  </div>
-</div>
+<h3>{{ subtitle }}</h3>
 
-<div class="row mt-4">
+<div class="row">
   <div class="col">
     {% if state_definitions %}
   <table class="table table-hover table-bordered align-middle">
diff --git a/dashboard/templates/base.html b/dashboard/templates/base.html
index 716db77..06f917e 100644
--- a/dashboard/templates/base.html
+++ b/dashboard/templates/base.html
@@ -219,13 +219,14 @@
             </div>
           </div>
 
-          <div class="d-flex flex-wrap gap-2 justify-content-end p-2 mb-2">
+          <div class="d-flex flex-wrap gap-2 justify-content-end m-2 mb-4">
           {% block actions %}
           {% endblock %}
           </div>
+          <div class= "mx-2">
           {% block content %}
           {% endblock content %}
-
+          </div>
         </main>
       </div>
     </div>
diff --git a/dashboard/templates/unassigned_devices.html b/dashboard/templates/unassigned_devices.html
index 7ddbd18..6a6094a 100644
--- a/dashboard/templates/unassigned_devices.html
+++ b/dashboard/templates/unassigned_devices.html
@@ -3,27 +3,32 @@
 {% load paginacion %}
 
 {% block actions %}
+
     {% if lot %}
-    <a href="{% url 'lot:documents' object.id %}" type="button" class="btn btn-green-admin">
+    <a href="{#% url 'lot:documents' object.id %#}" type="button" class="btn btn-green-admin btn-todo" data-bs-toggle="tooltip" title="NOT IMPLEMENTED. Menu for adding documents for the lot">
       <i class="bi bi-folder2"></i>
       {% trans 'Documents' %}
     </a>
     {% endif %}
-    <a href="{# url 'dashboard:exports' object.id #}" type="button" class="btn btn-todo" data-bs-toggle="tooltip" title="NOT IMPLEMENTED. This action tries to emulate what devicehub-teal did, which was related to opening a dialog where you can select different options for export the devices as csv for all selected devices" >
+        {% if lot %}
+    <a href="{% url 'lot:properties' object.id %}" type="button" class="btn btn-green-admin" >
+      <i class="bi bi-tag"></i>
+      {% trans 'Properties' %}
+    </a>
+    {% endif %}
+      <a href="{# url 'dashboard:exports' object.id #}" type="button" class="btn btn-todo" data-bs-toggle="tooltip" title="NOT IMPLEMENTED. This action tries to emulate what devicehub-teal did, which was related to opening a dialog where you can select different options for export the devices as csv for all selected devices" >
       <i class="bi bi-reply"></i>
       {% trans 'Exports' %}
     </a>
-    {% if lot %}
-    <a href="{% url 'lot:annotations' object.id %}" type="button" class="btn btn-green-admin" >
-      <i class="bi bi-tag"></i>
-      {% trans 'properties' %}
-    </a>
-    {% endif %}
+
 
 {% endblock%}
 
 {% block content %}
-<h3>{{ subtitle }}</h3>
+
+{% if lot.name %}
+<h3 class="text-muted"> <i class="bi bi-folder2-open me-2"></i>{% trans "Lot" %} {{ lot.name }}</h3>
+{% endif %}
 
 <div class="dataTable-container mt-4">
   <form method="post">
@@ -91,7 +96,7 @@
             {% elif dev.type == "SolidStateDrive" %}
               <i class="bi bi-device-ssd"></i>
             {% elif dev.type == "NetworkAdapter" %}
-              <i class="bi bi-ethernet"></i>
+              <i class="bi bi-pci-card-network"></i>
             {% elif dev.type == "Processor" %}
               <i class="bi bi-cpu"></i>
             {% elif dev.type == "RamModule" %}
@@ -104,8 +109,6 @@
               <i class="bi bi-battery"></i>
             {% elif dev.type == "Camera" %}
               <i class="bi bi-camera"></i>
-            {% else %}
-              <i class="bi bi-question-circle"></i>
             {% endif %}
              {{dev.type}}
           </td>
@@ -132,10 +135,8 @@
   </form>
 </div>
 
-<div class="row mt-3">
-  <div class="col">
+<div class="mt-5 d-flex align-items-center justify-content-center">
     {% render_pagination page total_pages limit %}
-  </div>
 </div>
 
 <script>
diff --git a/device/templates/details.html b/device/templates/details.html
index 040a145..49e6f2f 100644
--- a/device/templates/details.html
+++ b/device/templates/details.html
@@ -48,9 +48,8 @@
 {% endblock %}
 
 {% block content %}
-  <div class="col">
-    <h3>{{ object.shortid }}</h3>
-  </div>
+  <h3>{{ object.shortid }}</h3>
+
 
   <div class="row">
     <div class="col">
diff --git a/evidence/templates/ev_details.html b/evidence/templates/ev_details.html
index c9f579e..5accedb 100644
--- a/evidence/templates/ev_details.html
+++ b/evidence/templates/ev_details.html
@@ -12,9 +12,9 @@
 {% block content %}
 <div class="row">
 
-  <div class="col-md-12 mb-3">
+  <div class="col-md-12 mb-4">
     <div class="card d-flex flex-wrap">
-      <div class="card-header bg-light">
+      <div class="card-header">
         <strong class="card-title">Evidence
           <span class="text-muted" id="uuid">{{ object.uuid }}</span>
           <button class="btn btn-sm btn-outline-secondary ms-1" onclick="copyToClipboard()">
@@ -41,6 +41,10 @@
                     </div>
                   {% endif %}
 
+                   <!-- submit button -->
+                  <button class="btn btn-sm btn btn-outline-success" d-flex align-items-end gap-1" type="submit" name="submit_form2">
+                    <i class="bi bi-floppy2-fill"></i> {% translate 'Save' %}
+                  </button>
                   <!-- erase server switch -->
                   <div class="form-check form-switch m-0">
                     <input
@@ -52,7 +56,6 @@
                       {% if form2.erase_server.value %}
                         checked
                       {% endif %}>
-
                     <label class="form-check-label ps-1" for="{{ form2.erase_server.id_for_label }}">
                       {% if form2.erase_server.value %}
                         <i class="pe-1 bi bi-eraser"></i>
@@ -63,10 +66,7 @@
                       {% endif %}
                     </label>
                   </div>
-                  <!-- submit button -->
-                  <button class="btn btn-sm btn btn-outline-success" d-flex align-items-end gap-1" type="submit" name="submit_form2">
-                    <i class="bi bi-floppy2-fill"></i> {% translate 'Save' %}
-                  </button>
+
                 </form>
 
             </div>
diff --git a/evidence/templates/upload.html b/evidence/templates/upload.html
index d800485..63c5630 100644
--- a/evidence/templates/upload.html
+++ b/evidence/templates/upload.html
@@ -32,7 +32,7 @@
 
   
 {% load django_bootstrap5 %}
-<form role="form" method="post" enctype="multipart/form-data" class="needs-validation row d-flex justify-content-center " novalidate>
+<form role="form" method="post" enctype="multipart/form-data" class="row d-flex justify-content-center ">
   {% csrf_token %}
 
   <!-- Drag n drop -->
@@ -43,7 +43,7 @@
     {# support for both evidence file and spreadsheet forms #}
     {% if form.evidence_file %}
       {% bootstrap_form form exclude="evidence_file" alert_error_type="none" error_css_class="alert alert-danger" %}
-      <input type="file" id="file-input" name="{{ form.evidence_file.html_name }}" class="visually-hidden" {% if form.evidence_file.field.widget.attrs.multiple %}multiple{% endif %}>
+      <input type="file" id="file-input" name="{{ form.evidence_file.html_name }}" class="visually-hidden" {% if form.evidence_file.field.widget.attrs.multiple %}multiple required{% endif %}>
     {% elif form.file_import %}
       {% bootstrap_form form exclude="file_import" alert_error_type="none" error_css_class="alert alert-danger" %}
       <input type="file" id="file-input" name="{{ form.file_import.html_name }}" class="visually-hidden">
diff --git a/user/templates/panel.html b/user/templates/panel.html
index 4bd91a2..13b4262 100644
--- a/user/templates/panel.html
+++ b/user/templates/panel.html
@@ -55,7 +55,7 @@
         <h5 class="card-title mb-3">
           <i class="bi bi-gear me-2"></i> {% translate 'Settings File' %}
         </h5>
-        <p class="card-text flex-grow-1">{% translate 'Download a settings file for your account.' %}</p>
+        <p class="card-text flex-grow-1">{% translate 'Download a settings file for your Workbench.' %}</p>
         <div class="text-end">
           <a href="{% url 'user:settings' %}" class="btn btn-outline-dark btn-sm d-inline-flex align-items-center">
             <span class="me-2">{% translate 'Go' %}</span>