From 389908c67a9304831788342449786bf8aa111e18 Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Fri, 4 May 2018 17:51:55 +1000
Subject: [PATCH 1/3] Moved sidenav into block

---
 InvenTree/part/templates/part/part_app_base.html | 6 ++++++
 InvenTree/part/templates/part/part_base.html     | 2 +-
 InvenTree/templates/base.html                    | 5 ++++-
 InvenTree/templates/sidebar.html                 | 9 ---------
 4 files changed, 11 insertions(+), 11 deletions(-)
 create mode 100644 InvenTree/part/templates/part/part_app_base.html
 delete mode 100644 InvenTree/templates/sidebar.html

diff --git a/InvenTree/part/templates/part/part_app_base.html b/InvenTree/part/templates/part/part_app_base.html
new file mode 100644
index 0000000000..00f42b84a6
--- /dev/null
+++ b/InvenTree/part/templates/part/part_app_base.html
@@ -0,0 +1,6 @@
+{% extends "base.html" %}
+
+{% block sidenav %}
+<h3>Parts</h3>
+<div id='part-tree'></div>
+{% endblock %}
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/part_base.html b/InvenTree/part/templates/part/part_base.html
index 4b3f9ae6f4..b6736ed45b 100644
--- a/InvenTree/part/templates/part/part_base.html
+++ b/InvenTree/part/templates/part/part_base.html
@@ -1,4 +1,4 @@
-{% extends "base.html" %}
+{% extends "part/part_app_base.html" %}
 
 {% load static %}
 
diff --git a/InvenTree/templates/base.html b/InvenTree/templates/base.html
index 0aa57dd85d..b06b0ad37d 100644
--- a/InvenTree/templates/base.html
+++ b/InvenTree/templates/base.html
@@ -32,7 +32,10 @@ InvenTree
 
 <div class='main body wrapper'>
 
-{% include "sidebar.html" %}
+<div class='sidenav' id='sidenav'>
+{% block sidenav %}
+{% endblock %}
+</div>
 
 <div class="container container-fluid inventree-content" id='inventree-content'>
 {% block content %}
diff --git a/InvenTree/templates/sidebar.html b/InvenTree/templates/sidebar.html
deleted file mode 100644
index 6428e3d238..0000000000
--- a/InvenTree/templates/sidebar.html
+++ /dev/null
@@ -1,9 +0,0 @@
-<div class='sidenav' id='sidenav'>
-
-<div id='part-tree'></div>
-
-<hr>
-
-<div id='stock-tree'></div>
-
-</div>
\ No newline at end of file

From 4f63d1283765caa36884df0ada16b753b3ab73ca Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Fri, 4 May 2018 18:53:39 +1000
Subject: [PATCH 2/3] Consolidated 'Part' app views

- Improved templating
- Part tree open/close status stored to session
---
 InvenTree/part/templates/part/cat_link.html   |   1 +
 InvenTree/part/templates/part/category.html   | 157 ++++++++++++++++++
 .../part/templates/part/category_detail.html  |  93 -----------
 .../part/templates/part/category_parts.html   |  63 -------
 .../part/category_subcategories.html          |  11 --
 InvenTree/part/templates/part/index.html      |  77 ---------
 .../part/templates/part/part_app_base.html    |  24 ++-
 InvenTree/part/templates/part/part_base.html  |   4 +-
 .../part/templates/part/subcategories.html    |  27 +++
 InvenTree/part/views.py                       |   4 +-
 InvenTree/static/css/inventree.css            |   4 +-
 InvenTree/static/script/sidenav.js            |  43 +++++
 InvenTree/static/script/trees.js              |  20 ---
 InvenTree/templates/base.html                 |  27 +--
 14 files changed, 262 insertions(+), 293 deletions(-)
 create mode 100644 InvenTree/part/templates/part/category.html
 delete mode 100644 InvenTree/part/templates/part/category_detail.html
 delete mode 100644 InvenTree/part/templates/part/category_parts.html
 delete mode 100644 InvenTree/part/templates/part/category_subcategories.html
 delete mode 100644 InvenTree/part/templates/part/index.html
 create mode 100644 InvenTree/part/templates/part/subcategories.html
 delete mode 100644 InvenTree/static/script/trees.js

diff --git a/InvenTree/part/templates/part/cat_link.html b/InvenTree/part/templates/part/cat_link.html
index 02ec88c45c..181f10ebb6 100644
--- a/InvenTree/part/templates/part/cat_link.html
+++ b/InvenTree/part/templates/part/cat_link.html
@@ -1,6 +1,7 @@
 <div class='navigation'>
 <nav aria-label="breadcrumb">
   <ol class="breadcrumb">
+    <li><a href='#' id='toggle-part-tree'><b>+</b></a></li>
     <li class="breadcrumb-item{% if category is None %} active" aria-current="page{% endif %}"><a href="/part/">Parts</a></li>
 {% if category %}
 {% for path_item in category.parentpath %}
diff --git a/InvenTree/part/templates/part/category.html b/InvenTree/part/templates/part/category.html
new file mode 100644
index 0000000000..b287207a51
--- /dev/null
+++ b/InvenTree/part/templates/part/category.html
@@ -0,0 +1,157 @@
+{% extends "part/part_app_base.html" %}
+{% load static %}
+
+{% block content %}
+
+<div class='row'>
+<div class='col-sm-6'>
+    {% if category %}
+    <h3>{{ category.name }}</h3>
+    <p>{{ category.description }}</p>
+    {% else %}
+    <h3>Parts</h3>
+    {% endif %}
+</div>
+<div class='col-sm-6'>
+    <h3>
+    <div style='float: right;'>
+    <button class='btn btn-success' id='cat-create'>New Category</button>
+    {% if category %}
+    <div class="dropdown" style="float: right;">
+        <button class="btn btn-primary dropdown-toggle" type="button" data-toggle="dropdown">
+            Options
+            <span class="caret"></span>
+        </button>
+        <ul class="dropdown-menu">
+          <li><a href="#" id='cat-edit' title='Edit part category'>Edit</a></li>
+          <li><a href="#" id='cat-delete' title='Delete part category'>Delete</a></li>
+        </ul>
+    </div>
+    {% endif %}
+    </div>
+    </h3>
+</div>
+</div>
+
+{% if category %}
+{% include "part/subcategories.html" with children=category.children.all %}
+{% else %}
+{% include "part/subcategories.html" with children=children %}
+{% endif %}
+<hr>
+
+<table class='table table-striped table-condensed' id='part-table'>
+</table>
+
+{% endblock %}
+
+{% block js_ready %}
+{{ block.super }}
+
+    $("#cat-create").click(function() {
+        launchModalForm("#modal-form",
+                        "{% url 'category-create' %}",
+                        {
+                            follow: true,
+                            {% if category %}
+                            data: {
+                                category: {{ category.id }}
+                            }
+                            {% endif %}
+                        });
+    })
+
+    {% if category %}
+    $("#cat-edit").click(function () {
+        launchModalForm("#modal-form",
+                        "{% url 'category-edit' category.id %}",
+                        {
+                            reload: true
+                        },
+                       );
+        return false;
+    });
+
+    {% if category.parent %}
+    var redirect = "{% url 'category-detail' category.parent.id %}";
+    {% else %}
+    var redirect = "{% url 'part-index' %}";
+    {% endif %}
+
+    $('#cat-delete').click(function() {
+        launchDeleteForm('#modal-delete',
+                         "{% url 'category-delete' category.id %}",
+                         {
+                             redirect: redirect
+                         });
+    });
+
+    {% endif %}
+
+    $("#part-table").bootstrapTable({
+        sortable: true,
+        search: true,
+        sortName: 'description',
+        idField: 'pk',
+        method: 'get',
+        pagination: true,
+        rememberOrder: true,
+        {% if category %}
+        queryParams: function(p) {
+            return {
+                {% if category %}
+                category: {{ category.id }},
+                {% endif %}
+            }
+        },
+        {% endif %}
+        columns: [
+            {
+                checkbox: true,
+                title: 'Select',
+                searchable: false,
+            },
+            {
+                field: 'pk',
+                title: 'ID',
+                visible: false,
+            },
+            {
+                field: 'name',
+                title: 'Part',
+                sortable: true,
+                formatter: function(value, row, index, field) {
+                    return renderLink(value, row.url);
+                }
+            },
+            {
+                sortable: true,
+                field: 'description',
+                title: 'Description',
+            },
+            {% if category == None %}
+            {
+                sortable: true,
+                field: 'category',
+                title: 'Category',
+                formatter: function(value, row, index, field) {
+                    if (row.category) {
+                        return renderLink(row.category.name, row.category.url);
+                    }
+                    else {
+                        return '';
+                    }
+                }
+            },
+            {% endif %}
+            {
+                field: 'total_stock',
+                title: 'Stock',
+                searchable: false,
+                sortable: true,
+            }
+        ],
+        url: "{% url 'api-part-list' %}",
+    });
+
+{% endblock %}
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/category_detail.html b/InvenTree/part/templates/part/category_detail.html
deleted file mode 100644
index 2f810f6891..0000000000
--- a/InvenTree/part/templates/part/category_detail.html
+++ /dev/null
@@ -1,93 +0,0 @@
-{% extends "base.html" %}
-
-{% load static %}
-
-{% block content %}
-
-{% include "part/cat_link.html" with category=category %}
-
-<p>
-    <b>{{ category.name }}</b><br>
-    <i>{{ category.description }}</i>
-</p>
-
-{% if category.has_children %}
-<h4>Subcategories</h4>
-{% include "part/category_subcategories.html" with children=category.children.all %}
-{% endif %}
-
-{% if category.has_parts %}
-<table class='table table-striped table-condensed' id='part-table'>
-</table>
-{% endif %}
-
-<div class='container-fluid'>
-    <button type='button' class='btn btn-primary' id='create-cat'>
-        New Category
-    </button>
-    <button class="btn btn-info" id='edit-category'>Edit Category</button>
-    <button class="btn btn-success" id='create-part'>New Part</button>
-    <button class="btn btn-danger" id='delete-category'>Delete Category</button>
-
-    <button class='btn btn-primary' id='get-rows'>Do thing</button>
-</div>
-
-{% include 'modals.html' %}
-
-{% endblock %}
-
-{% block js_load %}
-<script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
-{% endblock %}
-{% block js_ready %}
-
-    $("#edit-category").click(function () {
-        launchModalForm("#modal-form",
-                        "{% url 'category-edit' category.id %}",
-                        {reload: true},
-                       );
-    });
-
-    {% if category.parent %}
-    var categoryRedirect = "{% url 'category-detail' category.parent.id %}";
-    {% else %}
-    var categoryRedirect = "{% url 'part-index' %}";
-    {% endif %}
-
-    $("#delete-category").click(function() {
-        launchDeleteForm("#modal-delete",
-                         "{% url 'category-delete' category.id %}",
-                         {
-                             redirect: categoryRedirect
-                         });
-    });
-
-    $("#create-cat").click(function() {
-        launchModalForm("#modal-form",
-                        "{% url 'category-create' %}",
-                        {
-                            follow: true,
-                            data: {
-                                category: {{ category.id }}
-                            }
-                        });
-    });
-
-    $("#create-part").click( function() {
-        launchModalForm("#modal-form",
-                        "{% url 'part-create' %}",
-                        {
-                            data: {
-                                category: {{ category.id }}
-                            },
-                            follow: true
-                        });
-    });
-
-    {% include "part/category_parts.html" with category=category %}
-
-    $("#get-rows").click( function() {
-        alert($("#part-table").bootstrapTable('getSelections'));
-    });
-
-{% endblock %}
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/category_parts.html b/InvenTree/part/templates/part/category_parts.html
deleted file mode 100644
index f389fee83a..0000000000
--- a/InvenTree/part/templates/part/category_parts.html
+++ /dev/null
@@ -1,63 +0,0 @@
-    $("#part-table").bootstrapTable({
-        sortable: true,
-        search: true,
-        sortName: 'description',
-        idField: 'pk',
-        method: 'get',
-        pagination: true,
-        rememberOrder: true,
-        {% if category %}
-        queryParams: function(p) {
-            return {
-                category: {{ category.id }},
-            }
-        },
-        {% endif %}
-        columns: [
-            {
-                checkbox: true,
-                title: 'Select',
-                searchable: false,
-            },
-            {
-                field: 'pk',
-                title: 'ID',
-                visible: false,
-            },
-            {
-                field: 'name',
-                title: 'Part',
-                sortable: true,
-                formatter: function(value, row, index, field) {
-                    return renderLink(value, row.url);
-                }
-            },
-            {
-                sortable: true,
-                field: 'description',
-                title: 'Description',
-            },
-            {% if category == None %}
-            {
-                sortable: true,
-                field: 'category',
-                title: 'Category',
-                formatter: function(value, row, index, field) {
-                    if (row.category) {
-                        return renderLink(row.category.name, row.category.url);
-                    }
-                    else {
-                        return '';
-                    }
-                }
-            },
-            {% endif %}
-            {
-                field: 'total_stock',
-                title: 'Stock',
-                searchable: false,
-                sortable: true,
-            }
-        ],
-        url: "{% url 'api-part-list' %}",
-    });
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/category_subcategories.html b/InvenTree/part/templates/part/category_subcategories.html
deleted file mode 100644
index 6b7f6e56a2..0000000000
--- a/InvenTree/part/templates/part/category_subcategories.html
+++ /dev/null
@@ -1,11 +0,0 @@
-<ul class="list-group">
-{% for child in children %}
-<li class="list-group-item">
-    <b><a href="{% url 'category-detail' child.id %}">{{ child.name }}</a></b>
-    {% if child.description %}
-    <i> - {{ child.description }}</i>
-    {% endif %}
-    <span class='badge'>{{ child.partcount }}</span>
-</li>
-{% endfor %}
-</ul>
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/index.html b/InvenTree/part/templates/part/index.html
deleted file mode 100644
index f928733eb4..0000000000
--- a/InvenTree/part/templates/part/index.html
+++ /dev/null
@@ -1,77 +0,0 @@
-{% extends "base.html" %}
-{% load static %}
-
-{% block css %}
-<link rel='stylesheet' href="{% static 'css/bootstrap-treeview.css' %}">
-{% endblock %}
-
-{% block content %}
-
-{% include "part/cat_link.html" with category=category %}
-
-{% include 'modals.html' %}
-
-{% if children.all|length > 0 %}
-<h4>Part Categories</h4>
-{% include "part/category_subcategories.html" with children=children %}
-{% endif %}
-
-<table class='table table-striped table-condensed' id='part-table'>
-
-<div class='container-fluid'>
-<button type='button' class='btn btn-primary' id='create-cat'>
-    New Category
-</button>
-<button class="btn btn-success" id='create-part'>New Part</button>
-</div>
-{% endblock %}
-
-{% block js_load %}
-
-<script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
-{% endblock %}
-{% block js_ready %}
-
-    $("#create-cat").click(function() {
-        launchModalForm("#modal-form",
-                        "{% url 'category-create' %}",
-                        {
-                            follow: true
-                        });
-    });
-
-    function loadTree() {
-        var requestData = {};
-
-        {% if category %}
-        requestData.category = {{ category.id }};
-        {% endif %}
-
-        $.ajax({
-            url: "{% url 'api-part-tree' %}",
-            type: 'get',
-            dataType: 'json',
-            data: requestData,
-            success: function (response) {
-                if (response.tree) {
-                    $("#part-tree").treeview({
-                        data: response.tree,
-                        enableLinks: true
-                    });
-                }
-            },
-            error: function (xhr, ajaxOptions, thrownError) {
-                alert('Error retrieving part tree:\n' + thrownError);
-            }
-        });
-    }
-
-    $("#create-part").click(function() {
-        launchModalForm("#modal-form", "{% url 'part-create' %}");
-    });
-
-    {% include "part/category_parts.html" %}
-
-    loadTree();
-
-{% endblock %}
diff --git a/InvenTree/part/templates/part/part_app_base.html b/InvenTree/part/templates/part/part_app_base.html
index 00f42b84a6..30c9b30b6d 100644
--- a/InvenTree/part/templates/part/part_app_base.html
+++ b/InvenTree/part/templates/part/part_app_base.html
@@ -1,6 +1,28 @@
 {% extends "base.html" %}
 
 {% block sidenav %}
-<h3>Parts</h3>
 <div id='part-tree'></div>
+{% endblock %}
+
+{% block pre_content %}
+
+{% if part %}
+{% include "part/cat_link.html" with category=part.category %}
+{% else %}
+{% include 'part/cat_link.html' with category=category %}
+{% endif %}
+
+{% endblock %}
+
+{% block js_ready %}
+{{ block.super }}
+    loadTree("{% url 'api-part-tree' %}",
+             "#part-tree");
+
+    $("#toggle-part-tree").click(function() {
+        toggleSideNav("#sidenav");
+        return false;
+    })
+
+    initSideNav();
 {% endblock %}
\ No newline at end of file
diff --git a/InvenTree/part/templates/part/part_base.html b/InvenTree/part/templates/part/part_base.html
index b6736ed45b..8c622ff12a 100644
--- a/InvenTree/part/templates/part/part_base.html
+++ b/InvenTree/part/templates/part/part_base.html
@@ -4,8 +4,6 @@
 
 {% block content %}
 
-{% include "part/cat_link.html" with category=part.category %}
-
 <div class="row">
 <div class="col-sm-6">
     <div class="media">
@@ -85,7 +83,7 @@
 {% endblock %}
 
 {% block js_ready %}
-
+{{ block.super }}
     $("#part-thumb").click(function() {
         launchModalForm("#modal-form",
                         "{% url 'part-image' part.id %}",
diff --git a/InvenTree/part/templates/part/subcategories.html b/InvenTree/part/templates/part/subcategories.html
new file mode 100644
index 0000000000..4f8704e736
--- /dev/null
+++ b/InvenTree/part/templates/part/subcategories.html
@@ -0,0 +1,27 @@
+{% if children|length > 0 %}
+<hr>
+<div class="panel-group">
+ <div class="panel panel-default">
+   <div class="panel-heading">
+     <h4 class="panel-title">
+       <a data-toggle="collapse" href="#collapse1">Child Categories</a><span class='badge'>{{ children|length }}</span>
+     </h4>
+   </div>
+   <div id="collapse1" class="panel-collapse collapse">
+     <div class="panel-body">
+         <ul class="list-group">
+         {% for child in children %}
+         <li class="list-group-item">
+             <b><a href="{% url 'category-detail' child.id %}">{{ child.name }}</a></b>
+             {% if child.description %}
+             <i> - {{ child.description }}</i>
+             {% endif %}
+             <span class='badge'>{{ child.partcount }}</span>
+         </li>
+         {% endfor %}
+         </ul>
+     </div>
+   </div>
+ </div>
+</div>
+{% endif %}
\ No newline at end of file
diff --git a/InvenTree/part/views.py b/InvenTree/part/views.py
index f11a328af2..9bb42722eb 100644
--- a/InvenTree/part/views.py
+++ b/InvenTree/part/views.py
@@ -23,7 +23,7 @@ from InvenTree.views import AjaxCreateView, AjaxUpdateView, AjaxDeleteView
 
 class PartIndex(ListView):
     model = Part
-    template_name = 'part/index.html'
+    template_name = 'part/category.html'
     context_object_name = 'parts'
 
     def get_queryset(self):
@@ -128,7 +128,7 @@ class CategoryDetail(DetailView):
     model = PartCategory
     context_object_name = 'category'
     queryset = PartCategory.objects.all()
-    template_name = 'part/category_detail.html'
+    template_name = 'part/category.html'
 
 
 class CategoryEdit(AjaxUpdateView):
diff --git a/InvenTree/static/css/inventree.css b/InvenTree/static/css/inventree.css
index e4f01e97bf..70cee55d51 100644
--- a/InvenTree/static/css/inventree.css
+++ b/InvenTree/static/css/inventree.css
@@ -47,7 +47,7 @@
     margin-right: 50px;
     margin-left: 50px;
     width: 100%;
-    transition: 0.5s;
+    transition: 0.1s;
 }
 
 .body {
@@ -88,7 +88,7 @@
     position: fixed; /* Stay in place */
     background-color: #fff; /* Black*/
     overflow-x: hidden; /* Disable horizontal scroll */
-    transition: 0.5s; /* 0.5 second transition effect to slide in the sidenav */
+    transition: 0.1s; /* 0.5 second transition effect to slide in the sidenav */
 }
 
 .wrapper {
diff --git a/InvenTree/static/script/sidenav.js b/InvenTree/static/script/sidenav.js
index 1520b91ea7..382a126148 100644
--- a/InvenTree/static/script/sidenav.js
+++ b/InvenTree/static/script/sidenav.js
@@ -1,9 +1,52 @@
+function loadTree(url, tree, data) {
+
+    $.ajax({
+        url: url,
+        type: 'get',
+        dataType: 'json',
+        data: data,
+        success: function (response) {
+            if (response.tree) {
+                $(tree).treeview({
+                    data: response.tree,
+                    enableLinks: true
+                });
+            }
+        },
+        error: function (xhr, ajaxOptions, thrownError) {
+            //TODO
+        }
+    });
+}
+
 function openSideNav() {
     document.getElementById("sidenav").style.width = "250px";
     document.getElementById("inventree-content").style.marginLeft = "270px";
+
+    sessionStorage.setItem('inventree-sidenav-state', 'open');
 }
 
 function closeSideNav() {
     document.getElementById("sidenav").style.width = "0";
     document.getElementById("inventree-content").style.marginLeft = "50px";
+
+    sessionStorage.setItem('inventree-sidenav-state', 'closed');
+}
+
+function toggleSideNav(nav) {
+    if ($(nav).width() == 0) {
+        openSideNav();
+    }
+    else {
+        closeSideNav();
+    }
+}
+
+function initSideNav() {
+    if (sessionStorage.getItem("inventree-sidenav-state") && sessionStorage.getItem('inventree-sidenav-state') == 'open') {
+        openSideNav();
+    }
+    else {
+        closeSideNav();
+    }
 }
\ No newline at end of file
diff --git a/InvenTree/static/script/trees.js b/InvenTree/static/script/trees.js
deleted file mode 100644
index 2938f3366e..0000000000
--- a/InvenTree/static/script/trees.js
+++ /dev/null
@@ -1,20 +0,0 @@
-function loadTree(url, tree, data) {
-
-    $.ajax({
-        url: url,
-        type: 'get',
-        dataType: 'json',
-        data: data,
-        success: function (response) {
-            if (response.tree) {
-                $(tree).treeview({
-                    data: response.tree,
-                    enableLinks: true
-                });
-            }
-        },
-        error: function (xhr, ajaxOptions, thrownError) {
-            //TODO
-        }
-    });
-}
\ No newline at end of file
diff --git a/InvenTree/templates/base.html b/InvenTree/templates/base.html
index b06b0ad37d..32d928306b 100644
--- a/InvenTree/templates/base.html
+++ b/InvenTree/templates/base.html
@@ -38,11 +38,16 @@ InvenTree
 </div>
 
 <div class="container container-fluid inventree-content" id='inventree-content'>
+{% block pre_content %}
+{% endblock %}
 {% block content %}
 <!-- Each view fills in here.. -->
 {% endblock %}
+{% block post_content %}
+{% endblock %}
 </div>
 
+{% include 'modals.html' %}
 {% include 'notification.html' %}
 </div>
 
@@ -57,38 +62,18 @@ InvenTree
 <script type='text/javascript' src="{% static 'script/bootstrap-table-en-US.min.js' %}"></script>
 
 <script type='text/javascript' src="{% static 'script/tables.js' %}"></script>
-<script type='text/javascript' src="{% static 'script/trees.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/sidenav.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/notification.js' %}"></script>
 <script type='text/javascript' src="{% static 'script/jquery.form.min.js' %}"></script>
-
+<script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
 {% block js_load %}
 {% endblock %}
 
 <script type='text/javascript'>
 $(document).ready(function () {
-
-    loadTree("{% url 'api-part-tree' %}",
-             "#part-tree");
-
-    loadTree("{% url 'api-stock-tree' %}",
-             "#stock-tree");
-
-    $('#logo').click(function() {
-        if ($("#sidenav").width() == 0) {
-            openSideNav();
-        }
-        else {
-            closeSideNav();
-        }
-    });
-
 {% block js_ready %}
-
 {% endblock %}
-
     showCachedAlerts();
-
 });
 </script>
 

From a7d4e299b233399f9d3e773e66b8cba3bb607a07 Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Fri, 4 May 2018 19:28:28 +1000
Subject: [PATCH 3/3] Cleaned up stock app

---
 InvenTree/static/css/inventree.css            |  4 +-
 InvenTree/stock/templates/stock/index.html    | 47 ---------
 InvenTree/stock/templates/stock/item.html     |  6 +-
 InvenTree/stock/templates/stock/loc_link.html |  1 +
 InvenTree/stock/templates/stock/location.html | 97 ++++++++++++++++---
 .../stock/templates/stock/location_list.html  | 28 +++++-
 .../stock/templates/stock/stock_app_base.html | 27 ++++++
 .../stock/templates/stock/stock_table.html    | 60 ------------
 InvenTree/stock/views.py                      |  2 +-
 9 files changed, 139 insertions(+), 133 deletions(-)
 delete mode 100644 InvenTree/stock/templates/stock/index.html
 create mode 100644 InvenTree/stock/templates/stock/stock_app_base.html
 delete mode 100644 InvenTree/stock/templates/stock/stock_table.html

diff --git a/InvenTree/static/css/inventree.css b/InvenTree/static/css/inventree.css
index 70cee55d51..e23039a58e 100644
--- a/InvenTree/static/css/inventree.css
+++ b/InvenTree/static/css/inventree.css
@@ -47,7 +47,7 @@
     margin-right: 50px;
     margin-left: 50px;
     width: 100%;
-    transition: 0.1s;
+    //transition: 0.1s;
 }
 
 .body {
@@ -88,7 +88,7 @@
     position: fixed; /* Stay in place */
     background-color: #fff; /* Black*/
     overflow-x: hidden; /* Disable horizontal scroll */
-    transition: 0.1s; /* 0.5 second transition effect to slide in the sidenav */
+    //transition: 0.1s; /* 0.5 second transition effect to slide in the sidenav */
 }
 
 .wrapper {
diff --git a/InvenTree/stock/templates/stock/index.html b/InvenTree/stock/templates/stock/index.html
deleted file mode 100644
index 2409af6c71..0000000000
--- a/InvenTree/stock/templates/stock/index.html
+++ /dev/null
@@ -1,47 +0,0 @@
-{% extends "base.html" %}
-{% load static %}
-{% block content %}
-
-{% include "stock/loc_link.html" with location=None %}
-
-<div class='row'>
-    <div class='col-sm-6'>
-        <h3>Storage Locations</h3>
-    </div>
-    <div class='col-sm-6'>
-        <h3>
-            <button style='float: right;' class='btn btn-success' id='location-create'>New Stock Location</button>
-        </h3>
-    </div>
-</div>
-
-<hr>
-
-{% if locations|length > 0 %}
-{% include "stock/location_list.html" with locations=locations %}
-<hr>
-{% endif %}
-<h3>Stock Items</h3>
-<table class="table table-striped" id='stock-table'>
-</table>
-
-{% include 'modals.html' %}
-
-{% endblock %}
-
-{% block js_load %}
-<script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
-{% endblock %}
-{% block js_ready %}
-
-    $('#location-create').click(function () {
-        launchModalForm("#modal-form",
-                        "{% url 'stock-location-create' %}",
-                        {
-                            follow: true
-                        });
-    });
-
-    {% include "stock/stock_table.html" %}
-
-{% endblock %}
\ No newline at end of file
diff --git a/InvenTree/stock/templates/stock/item.html b/InvenTree/stock/templates/stock/item.html
index efb703976c..50a839f52c 100644
--- a/InvenTree/stock/templates/stock/item.html
+++ b/InvenTree/stock/templates/stock/item.html
@@ -1,9 +1,7 @@
-{% extends "base.html" %}
+{% extends "stock/stock_app_base.html" %}
 {% load static %}
 {% block content %}
 
-{% include "stock/loc_link.html" with location=item.location %}
-
 <h3>Stock entry details</h3>
 
 <table class="table table-striped">
@@ -109,8 +107,6 @@
 <button class='btn btn-danger' id='delete-item'>Delete Stock Item</button>
 </div>
 
-{% include 'modals.html' %}
-
 {% endblock %}
 
 {% block js_load %}
diff --git a/InvenTree/stock/templates/stock/loc_link.html b/InvenTree/stock/templates/stock/loc_link.html
index 34f10f33f7..7da211e58c 100644
--- a/InvenTree/stock/templates/stock/loc_link.html
+++ b/InvenTree/stock/templates/stock/loc_link.html
@@ -1,6 +1,7 @@
 <div class="navigation">
 <nav aria-label="breadcrumb">
   <ol class="breadcrumb">
+    <li><a href='#' id='toggle-stock-tree'><b>+</b></a></li>  
     <li class="breadcrumb-item{% if location is None %} active" aria-current="page{% endif %}"><a href="/stock/">Stock</a></li>
 {% if location %}
 {% for path_item in location.parentpath %}
diff --git a/InvenTree/stock/templates/stock/location.html b/InvenTree/stock/templates/stock/location.html
index 2fc6cbd523..1ef14f2d82 100644
--- a/InvenTree/stock/templates/stock/location.html
+++ b/InvenTree/stock/templates/stock/location.html
@@ -1,45 +1,50 @@
-{% extends "base.html" %}
+{% extends "stock/stock_app_base.html" %}
 {% load static %}
 {% block content %}
 
-{% include "stock/loc_link.html" with location=location %}
-
 <div class='row'>
 <div class='col-sm-6'>
+    {% if location %}
     <h3>{{ location.name }}</h3>
     <p>{{ location.description }}</p>
+    {% else %}
+    <h3>Stock</h3>
+    {% endif %}
 </div>
 <div class='col-sm-6'>
     <h3>
     <div style='float: right;'>
     <button class='btn btn-success' id='location-create'>New Stock Location</button>
+    {% if location %}
     <div class="dropdown" style="float: right;">
-        <button class="btn btn-primary dropdown-toggle" type="button" data-toggle="dropdown">Stock Location
+        <button class="btn btn-primary dropdown-toggle" type="button" data-toggle="dropdown">Options
         <span class="caret"></span></button>
         <ul class="dropdown-menu">
           <li><a href="#" id='location-edit' title='Edit stock location'>Edit</a></li>
           <li><a href="#" id='location-delete' title='Delete stock location'>Delete</a></li>
         </ul>
     </div>
+    {% endif %}
     </div>
     </h3>
 </div>
 </div>
 
-{% if location.has_children %}
-<h3>Sub Locations</h3>
-{% include "stock/location_list.html" with locations=location.children %}
+{% if location %}
+{% include 'stock/location_list.html' with children=location.children.all %}
+{% else %}
+{% include 'stock/location_list.html' with children=locations %}
 {% endif %}
 
-{% if location.has_items %}
+<hr>
+
 <table class='table table-striped table-condensed' id='stock-table'>
 </table>
-{% endif %}
 
 <div class='container-fluid' style='float: right;'>
     <button class="btn btn-success" id='item-create'>New Stock Item</span></button>
     <div class="dropdown" style='float: right;'>
-        <button class="btn btn-primary dropdown-toggle" type="button" data-toggle="dropdown">Selected
+        <button class="btn btn-primary dropdown-toggle" type="button" data-toggle="dropdown">Options
         <span class="caret"></span></button>
         <ul class="dropdown-menu">
             <li><a href="#" id='multi-item-stocktake' title='Stocktake selected stock items'>Stocktake</a></li>
@@ -57,19 +62,22 @@
 <script type='text/javascript' src="{% static 'script/modal_form.js' %}"></script>
 {% endblock %}
 {% block js_ready %}
-
+{{ block.super }}
     $('#location-create').click(function () {
         launchModalForm("#modal-form",
                         "{% url 'stock-location-create' %}",
                         {
                             data: {
+                                {% if location %}
                                 location: {{ location.id }}
+                                {% endif %}
                             },
                             follow: true
                         });
         return false;
     });
 
+    {% if location %}
     $('#location-edit').click(function() {
         launchModalForm("#modal-form",
                         "{% url 'stock-location-edit' location.id %}",
@@ -86,7 +94,9 @@
                              redirect: "{% url 'stock-index' %}"
                          });
         return false;
-    })
+    });
+
+    {% endif %}
 
     $('#item-create').click(function () {
         launchModalForm("#modal-form",
@@ -96,7 +106,9 @@
                                 $("#stock-table").bootstrapTable('refresh');
                             },
                             data: {
+                                {% if location %}
                                 location: {{ location.id }}
+                                {% endif %}
                             }
                         });
 
@@ -133,6 +145,65 @@
         return false;
     });
 
-    {% include 'stock/stock_table.html' with location=location %}
+    $("#stock-table").bootstrapTable({
+        sortable: true,
+        search: true,
+        method: 'get',
+        pagination: true,
+        rememberOrder: true,
+        {% if location %}
+        queryParams: function(p) {
+            return {
+                location: {{ location.id }}
+            }
+        },
+        {% endif %}
+        columns: [
+            {
+                checkbox: true,
+                title: 'Select',
+                searchable: false,
+            },
+            {
+                field: 'pk',
+                title: 'ID',
+                visible: false,
+            },
+            {
+                field: 'part.name',
+                title: 'Part',
+                sortable: true,
+                formatter: function(value, row, index, field) {
+                    return renderLink(value, row.part.url);
+                }
+            },
+            {% if location == None %}
+            {
+                field: 'location',
+                title: 'Location',
+                sortable: true,
+                formatter: function(value, row, index, field) {
+                    if (row.location) {
+                        return renderLink(row.location.name, row.location.url);
+                    }
+                    else {
+                        return '';
+                    }
+                }
+            },
+            {% endif %}
+            {
+                field: 'quantity',
+                title: 'Stock',
+                sortable: true,
+            },
+            {
+                field: 'status',
+                title: 'Status',
+                sortable: true,
+            }
+        ],
+        url: "{% url 'api-stock-list' %}",
+    });
 
 {% endblock %}
diff --git a/InvenTree/stock/templates/stock/location_list.html b/InvenTree/stock/templates/stock/location_list.html
index ded73cfb3d..b906133e70 100644
--- a/InvenTree/stock/templates/stock/location_list.html
+++ b/InvenTree/stock/templates/stock/location_list.html
@@ -1,5 +1,23 @@
-<ul class="list-group">
-{% for child in locations.all %}
-<li class="list-group-item"><a href="{% url 'stock-location-detail' child.id %}">{{ child.name }}</a> - <i>{{ child.description }}</i></li>
-{% endfor %}
-</ul>
\ No newline at end of file
+{% if children|length > 0 %}
+<hr>
+<div class="panel-group">
+ <div class="panel panel-default">
+   <div class="panel-heading">
+     <h4 class="panel-title">
+       <a data-toggle="collapse" href="#collapse1">Sub-Locations</a><span class='badge'>{{ children|length }}</span>
+     </h4>
+   </div>
+   <div id="collapse1" class="panel-collapse collapse">
+     <div class="panel-body">
+         <ul class="list-group">
+         {% for child in children %}
+        <li class="list-group-item"><a href="{% url 'stock-location-detail' child.id %}">{{ child.name }}</a> - <i>{{ child.description }}</i></li>
+             <span class='badge'>{{ child.partcount }}</span>
+         </li>
+         {% endfor %}
+         </ul>
+     </div>
+   </div>
+ </div>
+</div>
+{% endif %}
\ No newline at end of file
diff --git a/InvenTree/stock/templates/stock/stock_app_base.html b/InvenTree/stock/templates/stock/stock_app_base.html
new file mode 100644
index 0000000000..5afe8e44dc
--- /dev/null
+++ b/InvenTree/stock/templates/stock/stock_app_base.html
@@ -0,0 +1,27 @@
+{% extends "base.html" %}
+{% load static %}
+
+{% block sidenav %}
+<div id='stock-tree'></div>
+{% endblock %}
+
+{% block pre_content %}
+{% if item %}
+{% include 'stock/loc_link.html' with location=item.location %}
+{% else %}
+{% include 'stock/loc_link.html' with location=location %}
+{% endif %}
+{% endblock %}
+
+{% block js_ready %}
+    initSideNav();
+{{ block.super }}
+    loadTree("{% url 'api-stock-tree' %}",
+             "#stock-tree");
+
+    $("#toggle-stock-tree").click(function() {
+        toggleSideNav("#sidenav");
+        return false;
+    })
+
+{% endblock %}
\ No newline at end of file
diff --git a/InvenTree/stock/templates/stock/stock_table.html b/InvenTree/stock/templates/stock/stock_table.html
deleted file mode 100644
index d9cb775ac1..0000000000
--- a/InvenTree/stock/templates/stock/stock_table.html
+++ /dev/null
@@ -1,60 +0,0 @@
-$("#stock-table").bootstrapTable({
-    sortable: true,
-    search: true,
-    method: 'get',
-    pagination: true,
-    rememberOrder: true,
-    {% if location %}
-    queryParams: function(p) {
-        return {
-            location: {{ location.id }}
-        }
-    },
-    {% endif %}
-    columns: [
-        {
-            checkbox: true,
-            title: 'Select',
-            searchable: false,
-        },
-        {
-            field: 'pk',
-            title: 'ID',
-            visible: false,
-        },
-        {
-            field: 'part.name',
-            title: 'Part',
-            sortable: true,
-            formatter: function(value, row, index, field) {
-                return renderLink(value, row.part.url);
-            }
-        },
-        {% if location == None %}
-        {
-            field: 'location',
-            title: 'Location',
-            sortable: true,
-            formatter: function(value, row, index, field) {
-                if (row.location) {
-                    return renderLink(row.location.name, row.location.url);
-                }
-                else {
-                    return '';
-                }
-            }
-        },
-        {% endif %}
-        {
-            field: 'quantity',
-            title: 'Stock',
-            sortable: true,
-        },
-        {
-            field: 'status',
-            title: 'Status',
-            sortable: true,
-        }
-    ],
-    url: "{% url 'api-stock-list' %}",
-});
\ No newline at end of file
diff --git a/InvenTree/stock/views.py b/InvenTree/stock/views.py
index 3a28b715d4..1b34312645 100644
--- a/InvenTree/stock/views.py
+++ b/InvenTree/stock/views.py
@@ -20,7 +20,7 @@ from .forms import StocktakeForm
 
 class StockIndex(ListView):
     model = StockItem
-    template_name = 'stock/index.html'
+    template_name = 'stock/location.html'
     context_obect_name = 'locations'
 
     def get_context_data(self, **kwargs):