Update tables to hide redundant fields

Closes #7

nummi/main/static/main/css/table.css

@@ -12,7 +12,9 @@
 .table.col6 {grid-template-columns: repeat(6, auto)}
 .table.col1-1 {grid-template-columns: min-content auto}
 .table.col1-1-1 {grid-template-columns: min-content auto min-content}
+.table.col2-4 {grid-template-columns: repeat(2, min-content) repeat(4, auto)}
 .table.col1-5 {grid-template-columns: min-content repeat(5, auto)}
+.table.col2-5 {grid-template-columns: repeat(2, min-content) repeat(5, auto)}
 .table.col1-6 {grid-template-columns: min-content repeat(6, auto)}
 .table.col1-7 {grid-template-columns: min-content repeat(7, auto)}
 

nummi/main/templates/main/table/snapshot.html

@@ -1,14 +1,17 @@
 {% load main_extras %}
 {% load i18n %}
-<div id="snapshots" class="table col1-6">
+<div id="snapshots"
+     class="table {% if account %}col2-4{% else %}col2-5{% endif %}">
     <div class="header">
+        <strong class="diff center"><i class="fa fa-check"></i></strong>
         <strong class="attach center"><i class="fa fa-paperclip"></i></strong>
         <strong class="date center">{% translate "Date" %}</strong>
+        {% if not account %}
             <strong class="account center">{% translate "Account" %}</strong>
+        {% endif %}
         <strong class="value center">{% translate "Value" %}</strong>
         <strong class="diff center">{% translate "Difference" %}</strong>
         <strong class="diff center">{% translate "Transactions" %}</strong>
-        <strong class="diff center">{% translate "Valid" %}</strong>
     </div>
     {% if new_snapshot_url %}
         <div class="full-line">
@@ -17,6 +20,13 @@
     {% endif %}
     {% for snap in snapshots %}
         <div class="snapshot">
+            <span class="valid center">
+                {% if snap.sum == snap.diff %}
+                    <i class="fa fa-check green"></i>
+                {% else %}
+                    <i class="fa fa-xmark red"></i>
+                {% endif %}
+            </span>
             <span class="attach center">
                 {% if snap.file %}
                     <a href="{{ snap.file.url }}">
@@ -27,22 +37,15 @@
             <span class="date num center">
                 <a href="{% url 'snapshot' pk=snap.id %}">{{ snap.date|date:"Y-m-d" }}</a>
             </span>
+            {% if not account %}
                 <span class="account text center">
                     <i class="fa fa-{{ snap.account.icon }}"></i>
                     <a href="{% url 'account' pk=snap.account.id %}">{{ snap.account }}</a>
                 </span>
+            {% endif %}
             <span class="value num right">{{ snap.value|value }}</span>
             <span class="diff num right">{{ snap.diff|pmvalue }}</span>
-            {% with sum=snap.sum %}
+            <span class="sum num right">{{ snap.sum|pmvalue }}</span>
-                <span class="sum num right">{{ sum|pmvalue }}</span>
-                <span class="valid center">
-                    {% if sum == snap.diff %}
-                        <i class="fa fa-check green"></i>
-                    {% else %}
-                        <i class="fa fa-xmark red"></i>
-                    {% endif %}
-                </span>
-            {% endwith %}
         </div>
     {% endfor %}
     {% if snapshots_url %}

nummi/main/templates/main/table/transaction.html

@@ -1,14 +1,19 @@
 {% load main_extras %}
 {% load i18n %}
-<div id="transactions" class="table col1-7">
+<div id="transactions"
+     class="table col1-{% if account or category %}6{% else %}7{% endif %}">
     <div class="header">
         <strong class="attach center"><i class="fa fa-paperclip"></i></strong>
         <strong class="date center">{% translate "Date" %}</strong>
         <strong class="name">{% translate "Name" %}</strong>
         <strong class="value center">{% translate "Value" %}</strong>
         <strong class="trader center">{% translate "Trader" %}</strong>
+        {% if not category %}
             <strong class="category center">{% translate "Category" %}</strong>
+        {% endif %}
+        {% if not account %}
             <strong class="account center">{% translate "Account" %}</strong>
+        {% endif %}
         <strong class="description">{% translate "Description" %}</strong>
     </div>
     {% if new_transaction_url %}
@@ -31,6 +36,7 @@
             <span class="name text"><a href="{% url 'transaction' pk=trans.id %}">{{ trans.name }}</a></span>
             <span class="value num right">{{ trans.value|pmvalue }}</span>
             <span class="trader text center">{{ trans.trader|default_if_none:"–" }}</span>
+            {% if not category %}
                 <span class="category text center">
                     {% if trans.category %}
                         <i class="fa fa-{{ trans.category.icon }}"></i>
@@ -39,6 +45,8 @@
                         –
                     {% endif %}
                 </span>
+            {% endif %}
+            {% if not account %}
                 <span class="account text center">
                     {% if trans.account %}
                         <i class="fa fa-{{ trans.account.icon }}"></i>
@@ -47,6 +55,7 @@
                         –
                     {% endif %}
                 </span>
+            {% endif %}
             <span class="description text">{{ trans.description }}</span>
         </div>
     {% endfor %}

nummi/main/views.py

@@ -288,7 +288,8 @@ class AccountMixin:
 
     def get_context_data(self, **kwargs):
         return super().get_context_data(**kwargs) | {
-            "object": Account.objects.get(pk=self.kwargs.get("pk"))
+            "object": Account.objects.get(pk=self.kwargs.get("pk")),
+            "account": True,
         }
 
 
@@ -298,7 +299,8 @@ class SnapshotMixin:
 
     def get_context_data(self, **kwargs):
         return super().get_context_data(**kwargs) | {
-            "object": Snapshot.objects.get(pk=self.kwargs.get("pk"))
+            "object": Snapshot.objects.get(pk=self.kwargs.get("pk")),
+            "snapshot": True,
         }
 
 
@@ -308,7 +310,8 @@ class CategoryMixin:
 
     def get_context_data(self, **kwargs):
         return super().get_context_data(**kwargs) | {
-            "object": Category.objects.get(pk=self.kwargs.get("pk"))
+            "object": Category.objects.get(pk=self.kwargs.get("pk")),
+            "category": True,
         }