From 2a7b32d4f42246f90a9b08d0936e42c380c8e022 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Wed, 24 Feb 2021 16:37:32 +1100
Subject: [PATCH] Part display tweaks

---
 InvenTree/InvenTree/static/css/inventree.css |   7 +-
 InvenTree/part/templates/part/part_base.html | 300 ++++++++++---------
 2 files changed, 157 insertions(+), 150 deletions(-)

diff --git a/InvenTree/InvenTree/static/css/inventree.css b/InvenTree/InvenTree/static/css/inventree.css
index fe3fc5e7cf..90210c6d70 100644
--- a/InvenTree/InvenTree/static/css/inventree.css
+++ b/InvenTree/InvenTree/static/css/inventree.css
@@ -545,7 +545,7 @@
 }
 
 .media {
-    padding-top: 15px;
+    //padding-top: 15px;
     overflow: visible;
 }
 
@@ -811,6 +811,11 @@ input[type="submit"] {
     background-color: #fafafa;
 }
 
+.panel-inventree {
+    padding: 10px;
+    box-shadow: 1px 1px #DDD;
+}
+
 .float-right {
     float: right;
 }
diff --git a/InvenTree/part/templates/part/part_base.html b/InvenTree/part/templates/part/part_base.html
index 8dd11bbf08..a2b7cc55bb 100644
--- a/InvenTree/part/templates/part/part_base.html
+++ b/InvenTree/part/templates/part/part_base.html
@@ -26,173 +26,175 @@
 </div>
 {% endif %}
 
-<div class="row">
-<div class="col-sm-6">
-    {% include "part/part_thumb.html" %}
-    <div class="media-body">
-        <h3>
-            {{ part.full_name }}
-            {% if user.is_staff and roles.part.change %}
-            <a  href="{% url 'admin:part_part_change' part.pk %}"><span title="{% trans 'Admin view' %}" class='fas fa-user-shield'></span></a>
-            {% endif %}
-            {% if not part.active %}
-            <div class='label label-large label-large-red'>
-                {% trans 'Inactive' %}
-            </div>
-            {% endif %}
-        </h3>   
-        <p><i>{{ part.description }}</i></p>
+<div class='panel panel-default panel-inventree'>
+    <div class="row">
+    <div class="col-sm-6">
+        {% include "part/part_thumb.html" %}
+        <div class="media-body">
+            <h3>
+                {{ part.full_name }}
+                {% if user.is_staff and roles.part.change %}
+                <a  href="{% url 'admin:part_part_change' part.pk %}"><span title="{% trans 'Admin view' %}" class='fas fa-user-shield'></span></a>
+                {% endif %}
+                {% if not part.active %}
+                <div class='label label-large label-large-red'>
+                    {% trans 'Inactive' %}
+                </div>
+                {% endif %}
+            </h3>   
+            <p><i>{{ part.description }}</i></p>
 
-        <div class='btn-group action-buttons' role='group'>
-            <button type='button' class='btn btn-default' id='toggle-starred' title='{% trans "Star this part" %}'>
-                <span id='part-star-icon' class='fas fa-star {% if starred %}icon-yellow{% endif %}'/>
-            </button>
+            <div class='btn-group action-buttons' role='group'>
+                <button type='button' class='btn btn-default' id='toggle-starred' title='{% trans "Star this part" %}'>
+                    <span id='part-star-icon' class='fas fa-star {% if starred %}icon-yellow{% endif %}'/>
+                </button>
 
-            {% if barcodes %}
-            <!-- Barcode actions menu -->
-            <div class='btn-group'>
-                <button id='barcode-options' title='{% trans "Barcode actions" %}' class='btn btn-default dropdown-toggle' type='button' data-toggle='dropdown'><span class='fas fa-qrcode'></span> <span class='caret'></span></button>
-                <ul class='dropdown-menu'>
-                    <li><a href='#' id='show-qr-code'><span class='fas fa-qrcode'></span> {% trans "Show QR Code" %}</a></li>
-                    <li><a href='#' id='print-label'><span class='fas fa-tag'></span> {% trans "Print Label" %}</a></li>
-                </ul>
+                {% if barcodes %}
+                <!-- Barcode actions menu -->
+                <div class='btn-group'>
+                    <button id='barcode-options' title='{% trans "Barcode actions" %}' class='btn btn-default dropdown-toggle' type='button' data-toggle='dropdown'><span class='fas fa-qrcode'></span> <span class='caret'></span></button>
+                    <ul class='dropdown-menu'>
+                        <li><a href='#' id='show-qr-code'><span class='fas fa-qrcode'></span> {% trans "Show QR Code" %}</a></li>
+                        <li><a href='#' id='print-label'><span class='fas fa-tag'></span> {% trans "Print Label" %}</a></li>
+                    </ul>
+                </div>
+                {% endif %}
+                {% if part.active %}
+                <button type='button' class='btn btn-default' id='price-button' title='{% trans "Show pricing information" %}'>
+                    <span id='part-price-icon' class='fas fa-dollar-sign'/>
+                </button>
+                {% if roles.stock.change %}
+                <button type='button' class='btn btn-default' id='part-count' title='{% trans "Count part stock" %}'>
+                    <span class='fas fa-clipboard-list'/>
+                </button>
+                {% endif %}
+                {% if part.purchaseable %}
+                {% if roles.purchase_order.add %}
+                <button type='button' class='btn btn-default' id='part-order' title='{% trans "Order part" %}'>
+                    <span id='part-order-icon' class='fas fa-shopping-cart'/>
+                </button>
+                {% endif %}
+                {% endif %}
+                {% endif %}
+                <!-- Part actions -->
+                {% if roles.part.add or roles.part.change or roles.part.delete %}
+                <div class='btn-group'>
+                    <button id='part-actions' title='{% trans "Part actions" %}' class='btn btn-default dropdown-toggle' type='button' data-toggle='dropdown'> <span class='fas fa-shapes'></span> <span class='caret'></span></button>
+                    <ul class='dropdown-menu'>
+                        {% if roles.part.add %}
+                        <li><a href='#' id='part-duplicate'><span class='fas fa-copy'></span> {% trans "Duplicate part" %}</a></li>
+                        {% endif %}
+                        {% if roles.part.change %}
+                        <li><a href='#' id='part-edit'><span class='fas fa-edit icon-blue'></span> {% trans "Edit part" %}</a></li>
+                        {% endif %}
+                        {% if not part.active and roles.part.delete %}
+                        <li><a href='#' id='part-delete'><span class='fas fa-trash-alt icon-red'></span> {% trans "Delete part" %}</a></li>
+                        {% endif %}
+                    </ul>
+                </div>
+                {% endif %}
             </div>
-            {% endif %}
-            {% if part.active %}
-            <button type='button' class='btn btn-default' id='price-button' title='{% trans "Show pricing information" %}'>
-                <span id='part-price-icon' class='fas fa-dollar-sign'/>
-            </button>
-            {% if roles.stock.change %}
-            <button type='button' class='btn btn-default' id='part-count' title='{% trans "Count part stock" %}'>
-                <span class='fas fa-clipboard-list'/>
-            </button>
-            {% endif %}
-            {% if part.purchaseable %}
-            {% if roles.purchase_order.add %}
-            <button type='button' class='btn btn-default' id='part-order' title='{% trans "Order part" %}'>
-                <span id='part-order-icon' class='fas fa-shopping-cart'/>
-            </button>
-            {% endif %}
-            {% endif %}
-            {% endif %}
-            <!-- Part actions -->
-            {% if roles.part.add or roles.part.change or roles.part.delete %}
-            <div class='btn-group'>
-                <button id='part-actions' title='{% trans "Part actions" %}' class='btn btn-default dropdown-toggle' type='button' data-toggle='dropdown'> <span class='fas fa-shapes'></span> <span class='caret'></span></button>
-                <ul class='dropdown-menu'>
-                    {% if roles.part.add %}
-                    <li><a href='#' id='part-duplicate'><span class='fas fa-copy'></span> {% trans "Duplicate part" %}</a></li>
-                    {% endif %}
-                    {% if roles.part.change %}
-                    <li><a href='#' id='part-edit'><span class='fas fa-edit icon-blue'></span> {% trans "Edit part" %}</a></li>
-                    {% endif %}
-                    {% if not part.active and roles.part.delete %}
-                    <li><a href='#' id='part-delete'><span class='fas fa-trash-alt icon-red'></span> {% trans "Delete part" %}</a></li>
-                    {% endif %}
-                </ul>
-            </div>
-            {% endif %}
+            <table class='table table-condensed'>
+                <col width='25'>
+                {% if part.IPN %}
+                <tr>
+                    <td></td>
+                    <td>{% trans "IPN" %}</td>
+                    <td>{{ part.IPN }}</td>
+                </tr>
+                {% endif %}
+                {% if part.link %}
+                <tr>
+                    <td><span class='fas fa-link'></span></td>
+                    <td>{% trans "External Link" %}</td>
+                    <td><a href="{{ part.link }}">{{ part.link }}</a></td>
+                </tr>
+                {% endif %}
+                <tr>
+                </tr>
+            </table>
         </div>
-        <table class='table table-condensed'>
+        </div>
+    </div>
+    <div class="col-sm-6">
+        <table class="table table-striped">
             <col width='25'>
-            {% if part.IPN %}
+            <tr>
+                <td><span class='fas fa-boxes'></span></td>
+                <td>
+                    <h4>{% trans "Available Stock" %}</h4>
+                </td>
+                <td><h4>{% decimal available %}{% if part.units %} {{ part.units }}{% endif %}</h4></td>
+            </tr>
+            <tr>
+                <td><span class='fas fa-map-marker-alt'></span></td>
+                <td>{% trans "In Stock" %}</td>
+                <td>{% include "part/stock_count.html" %}</td>
+            </tr>
+            {% if on_order > 0 %}
+            <tr>
+                <td><span class='fas fa-shopping-cart'></span></td>
+                <td>{% trans "On Order" %}</td>
+                <td>{% decimal on_order %}</td>
+            </tr>
+            {% endif %}
+            {% if required_build_order_quantity > 0 %}
+            <tr>
+                <td><span class='fas fa-clipboard-list'></span></td>
+                <td>{% trans "Required for Build Orders" %}</td>
+                <td>{% decimal required_build_order_quantity %}
+            </tr>
+            {% endif %}
+            {% if required_sales_order_quantity > 0 %}
+            <tr>
+                <td><span class='fas fa-clipboard-list'></span></td>
+                <td>{% trans "Required for Sales Orders" %}</td>
+                <td>{% decimal required_sales_order_quantity %}
+            </tr>
+            {% endif %}
+            {% if allocated > 0 %}
+            <tr>
+                <td><span class='fas fa-dolly'></span></td>
+                <td>{% trans "Allocated to Orders" %}</td>
+                <td>{% decimal allocated %}</td>
+            </tr>
+            {% endif %}
+            
+            {% if not part.is_template %}
+            {% if part.assembly %}
+            <tr>
+                <td><span class='fas fa-tools'></span></td>
+                <td colspan='2'>
+                    <b>{% trans "Build Status" %}</b>
+                </td>
+            </tr>
             <tr>
                 <td></td>
-                <td>{% trans "IPN" %}</td>
-                <td>{{ part.IPN }}</td>
+                <td>{% trans "Can Build" %}</td>
+                <td>{% decimal part.can_build %}</td>
+            </tr>
+            {% if quantity_being_built > 0 %}
+            <tr>
+                <td></td>
+                <td>{% trans "Building" %}</td>
+                <td>{% decimal quantity_being_built %}</td>
             </tr>
             {% endif %}
-            {% if part.link %}
-            <tr>
-                <td><span class='fas fa-link'></span></td>
-                <td>{% trans "External Link" %}</td>
-                <td><a href="{{ part.link }}">{{ part.link }}</a></td>
-            </tr>
             {% endif %}
-            <tr>
-            </tr>
+        {% endif %}
         </table>
     </div>
     </div>
-</div>
-<div class="col-sm-6">
-    <table class="table table-striped">
-        <col width='25'>
-        <tr>
-            <td><span class='fas fa-boxes'></span></td>
-            <td>
-                <h4>{% trans "Available Stock" %}</h4>
-            </td>
-            <td><h4>{% decimal available %}{% if part.units %} {{ part.units }}{% endif %}</h4></td>
-        </tr>
-        <tr>
-            <td><span class='fas fa-map-marker-alt'></span></td>
-            <td>{% trans "In Stock" %}</td>
-            <td>{% include "part/stock_count.html" %}</td>
-        </tr>
-        {% if on_order > 0 %}
-        <tr>
-            <td><span class='fas fa-shopping-cart'></span></td>
-            <td>{% trans "On Order" %}</td>
-            <td>{% decimal on_order %}</td>
-        </tr>
-        {% endif %}
-        {% if required_build_order_quantity > 0 %}
-        <tr>
-            <td><span class='fas fa-clipboard-list'></span></td>
-            <td>{% trans "Required for Build Orders" %}</td>
-            <td>{% decimal required_build_order_quantity %}
-        </tr>
-        {% endif %}
-        {% if required_sales_order_quantity > 0 %}
-        <tr>
-            <td><span class='fas fa-clipboard-list'></span></td>
-            <td>{% trans "Required for Sales Orders" %}</td>
-            <td>{% decimal required_sales_order_quantity %}
-        </tr>
-        {% endif %}
-        {% if allocated > 0 %}
-        <tr>
-            <td><span class='fas fa-dolly'></span></td>
-            <td>{% trans "Allocated to Orders" %}</td>
-            <td>{% decimal allocated %}</td>
-        </tr>
-        {% endif %}
         
-        {% if not part.is_template %}
-        {% if part.assembly %}
-        <tr>
-            <td><span class='fas fa-tools'></span></td>
-            <td colspan='2'>
-                <b>{% trans "Build Status" %}</b>
-            </td>
-        </tr>
-        <tr>
-            <td></td>
-            <td>{% trans "Can Build" %}</td>
-            <td>{% decimal part.can_build %}</td>
-        </tr>
-        {% if quantity_being_built > 0 %}
-        <tr>
-            <td></td>
-            <td>{% trans "Building" %}</td>
-            <td>{% decimal quantity_being_built %}</td>
-        </tr>
-        {% endif %}
-        {% endif %}
-    {% endif %}
-    </table>
-</div>
 </div>
 
-<hr>
 <div class='container-fluid'>
-
+    
     {% block details %}
-
-<!-- Specific part details go here... -->
-
-{% endblock %}
+    
+    <!-- Specific part details go here... -->
+    
+    {% endblock %}
 </div>
 
 {% endblock %}