From 3849ea8057b886f8a37f7f7ce8d15038daf73d14 Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Fri, 29 Oct 2021 23:20:09 +1100
Subject: [PATCH] Rearranging buttons

---
 InvenTree/build/templates/build/detail.html   |  8 +-
 .../order/purchase_order_detail.html          |  8 +-
 .../templates/order/sales_order_detail.html   |  8 +-
 InvenTree/part/templates/part/detail.html     | 77 +++++++++++++------
 InvenTree/stock/templates/stock/item.html     | 41 ++++++----
 InvenTree/templates/attachment_button.html    |  5 ++
 InvenTree/templates/attachment_table.html     |  6 +-
 7 files changed, 110 insertions(+), 43 deletions(-)
 create mode 100644 InvenTree/templates/attachment_button.html

diff --git a/InvenTree/build/templates/build/detail.html b/InvenTree/build/templates/build/detail.html
index f2a16613b8..556c54f35c 100644
--- a/InvenTree/build/templates/build/detail.html
+++ b/InvenTree/build/templates/build/detail.html
@@ -275,7 +275,13 @@
 
 <div class='panel panel-hidden' id='panel-attachments'>
     <div class='panel-heading'>
-        <h4>{% trans "Attachments" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Attachments" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                {% include "attachment_button.html" %}
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         {% include "attachment_table.html" %}
diff --git a/InvenTree/order/templates/order/purchase_order_detail.html b/InvenTree/order/templates/order/purchase_order_detail.html
index c57d88fe57..6a016d110a 100644
--- a/InvenTree/order/templates/order/purchase_order_detail.html
+++ b/InvenTree/order/templates/order/purchase_order_detail.html
@@ -58,7 +58,13 @@
 
 <div class='panel panel-hidden' id='panel-order-attachments'>
     <div class='panel-heading'>
-        <h4>{% trans "Purchase Order Attachments" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Attachments" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                {% include "attachment_button.html" %}
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         {% include "attachment_table.html" %}
diff --git a/InvenTree/order/templates/order/sales_order_detail.html b/InvenTree/order/templates/order/sales_order_detail.html
index e5f047b520..5d4db70a34 100644
--- a/InvenTree/order/templates/order/sales_order_detail.html
+++ b/InvenTree/order/templates/order/sales_order_detail.html
@@ -49,7 +49,13 @@
 
 <div class='panel panel-hidden' id='panel-order-attachments'>
     <div class='panel-heading'>
-        <h4>{% trans "Attachments" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Attachments" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                {% include "attachment_button.html" %}
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         {% include "attachment_table.html" %}
diff --git a/InvenTree/part/templates/part/detail.html b/InvenTree/part/templates/part/detail.html
index 638482f9e3..d26915a628 100644
--- a/InvenTree/part/templates/part/detail.html
+++ b/InvenTree/part/templates/part/detail.html
@@ -129,14 +129,19 @@
 
 <div class='panel panel-hidden' id='panel-purchase-orders'>
     <div class='panel-heading'>
-        <h4>{% trans "Purchase Orders" %}</h4>
+        <div class='d-flex flex-row'>    
+            <h4>{% trans "Purchase Orders" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                <button class='btn btn-primary' type='button' id='part-order2' title='{% trans "Order part" %}'>
+                    <span class='fas fa-shopping-cart'></span> {% trans "Order Part" %}
+                </button>
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         <div id='po-button-bar'>
             <div class='button-toolbar container-fluid' style='float: right;'>
-                <button class='btn btn-primary' type='button' id='part-order2' title='{% trans "Order part" %}'>
-                    <span class='fas fa-shopping-cart'></span> {% trans "Order Part" %}
-                </button>
                 <div class='filter-list' id='filter-list-purchaseorder'>
                     <!-- An empty div in which the filter list will be constructed -->
                 </div>
@@ -234,11 +239,10 @@
 
 <div class='panel panel-hidden' id='panel-part-parameters'>
     <div class='panel-heading'>
-        <h4>{% trans "Parameters" %}</h4>
-    </div>
-    <div class='panel-content'>
-        <div id='param-button-toolbar'>
-            <div class='button-toolbar container-fluid' style='float: right;'>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Parameters" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
                 {% if roles.part.add %}
                 <button title='{% trans "Add new parameter" %}' class='btn btn-success' id='param-create'>
                     <span class='fas fa-plus-circle'></span> {% trans "New Parameter" %}
@@ -246,13 +250,25 @@
                 {% endif %}
             </div>
         </div>
+    </div>
+    <div class='panel-content'>
+        <div id='param-button-toolbar'>
+            <div class='button-toolbar container-fluid' style='float: right;'>
+            </div>
+        </div>
         <table id='parameter-table' class='table table-condensed table-striped' data-toolbar="#param-button-toolbar"></table>
     </div>
 </div>
 
 <div class='panel panel-hidden' id='panel-part-attachments'>
     <div class='panel-heading'>
-        <h4>{% trans "Attachments" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Attachments" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                {% include "attachment_button.html" %}
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         {% include "attachment_table.html" %}
@@ -261,17 +277,24 @@
 
 <div class='panel panel-hidden' id='panel-related-parts'>
     <div class='panel-heading'>
-        <h4>{% trans "Related Parts" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Related Parts" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                {% if roles.part.change %}
+                <button class='btn btn-success' type='button' id='add-related-part' title='{% trans "Add Related" %}'>
+                    <span class='fas fa-plus-circle'></span> {% trans "Add Related" %}
+                </button>
+                {% endif %}
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         <div id='related-button-bar'>
             <div class='button-toolbar container-fluid' style='float: left;'>
-                {% if roles.part.change %}
-                <button class='btn btn-success' type='button' id='add-related-part' title='{% trans "Add Related" %}'>{% trans "Add Related" %}</button>
                 <div class='filter-list' id='filter-list-related'>
                     <!-- An empty div in which the filter list will be constructed -->
                 </div>
-                {% endif %}
             </div>
         </div>
         
@@ -362,14 +385,19 @@
 
 <div class='panel panel-hidden' id='panel-suppliers'>
     <div class='panel-heading'>
-        <h4>{% trans "Part Suppliers" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Part Suppliers" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                <button class="btn btn-success" id='supplier-create'>
+                    <span class='fas fa-plus-circle'></span> {% trans "New Supplier Part" %}
+                </button>
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         <div id='supplier-button-toolbar'>
             <div class='btn-group'>
-                <button class="btn btn-success" id='supplier-create'>
-                    <span class='fas fa-plus-circle'></span> {% trans "New Supplier Part" %}
-                </button>
                 <div id='opt-dropdown' class="btn-group">
                     <button id='supplier-part-options' class="btn btn-primary dropdown-toggle" type="button" data-bs-toggle="dropdown">{% trans "Options" %} <span class="caret"></span></button>
                     <ul class="dropdown-menu">
@@ -384,15 +412,20 @@
     </div>
 
     <div class='panel-heading'>
-        <h4>{% trans "Part Manufacturers" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Part Manufacturers" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                <button class="btn btn-success" id='manufacturer-create'>
+                    <span class='fas fa-plus-circle'></span> {% trans "New Manufacturer Part" %}
+                </button>
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         <div class='panel-content'>
             <div id='manufacturer-button-toolbar'>
                 <div class='btn-group'>
-                    <button class="btn btn-success" id='manufacturer-create'>
-                        <span class='fas fa-plus-circle'></span> {% trans "New Manufacturer Part" %}
-                    </button>
                     <div id='opt-dropdown' class="btn-group">
                         <button id='manufacturer-part-options' class="btn btn-primary dropdown-toggle" type="button" data-bs-toggle="dropdown">{% trans "Options" %} <span class="caret"></span></button>
                         <ul class="dropdown-menu">
diff --git a/InvenTree/stock/templates/stock/item.html b/InvenTree/stock/templates/stock/item.html
index 554a8064e4..3da8d9da23 100644
--- a/InvenTree/stock/templates/stock/item.html
+++ b/InvenTree/stock/templates/stock/item.html
@@ -14,25 +14,30 @@
 
 <div class='panel panel-hidden' id='panel-history'>
     <div class='panel-heading'>
-        <h4>{% trans "Stock Tracking Information" %}</h4>
-    </div>
-    <div class='panel-content'>
-        {% setting_object 'STOCK_OWNERSHIP_CONTROL' as owner_control %}
-        {% if owner_control.value == "True" %}
-            {% authorized_owners item.owner as owners %}
-        {% endif %}
-        <!-- Check permissions and owner -->
-        {% if owner_control.value == "False" or owner_control.value == "True" and user in owners %}
-        {% if roles.stock.change and not item.is_building %}
-        <div id='table-toolbar'>  
-            <div class='btn-group'>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Stock Tracking Information" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                {% setting_object 'STOCK_OWNERSHIP_CONTROL' as owner_control %}
+                {% if owner_control.value == "True" %}
+                    {% authorized_owners item.owner as owners %}
+                {% endif %}
+                <!-- Check permissions and owner -->
+                {% if owner_control.value == "False" or owner_control.value == "True" and user in owners %}
+                {% if roles.stock.change and not item.is_building %}
                 <button class='btn btn-success' type='button' title='New tracking entry' id='new-entry'>
                     <span class='fas fa-plus-circle'></span> {% trans "New Entry" %}
                 </button>
+                {% endif %}
+                {% endif %}
+            </div>
+        </div>
+    </div>
+    <div class='panel-content'>
+        <div id='table-toolbar'>  
+            <div class='btn-group'>
             </div>
         </div>
-        {% endif %}
-        {% endif %}
         <table class='table table-condensed table-striped' id='track-table' data-toolbar='#table-toolbar'>
         </table>
     </div>
@@ -85,7 +90,13 @@
 
 <div class='panel panel-hidden' id='panel-attachments'>
     <div class='panel-heading'>
-        <h4>{% trans "Attachments" %}</h4>
+        <div class='d-flex flex-row'>
+            <h4>{% trans "Attachments" %}</h4>
+            {% include "spacer.html" %}
+            <div class='btn-group' role='group'>
+                {% include "attachment_button.html" %}
+            </div>
+        </div>
     </div>
     <div class='panel-content'>
         {% include "attachment_table.html" %}
diff --git a/InvenTree/templates/attachment_button.html b/InvenTree/templates/attachment_button.html
new file mode 100644
index 0000000000..e1561010c0
--- /dev/null
+++ b/InvenTree/templates/attachment_button.html
@@ -0,0 +1,5 @@
+{% load i18n %}
+
+<button type='button' class='btn btn-success' id='new-attachment'>
+    <span class='fas fa-plus-circle'></span> {% trans "Add Attachment" %}
+</button>
\ No newline at end of file
diff --git a/InvenTree/templates/attachment_table.html b/InvenTree/templates/attachment_table.html
index 18a4da9acc..c9b5a0bdf1 100644
--- a/InvenTree/templates/attachment_table.html
+++ b/InvenTree/templates/attachment_table.html
@@ -2,9 +2,9 @@
 
 <div id='attachment-buttons'>
     <div class='btn-group'>
-        <button type='button' class='btn btn-success' id='new-attachment'>
-            <span class='fas fa-plus-circle'></span> {% trans "Add Attachment" %}
-        </button>
+        <div class='filter-list' id='filter-list-related'>
+                    <!-- An empty div in which the filter list will be constructed -->
+        </div>
     </div>
 </div>