From 489c9b30eafd49c4af52b5b36c70a076ea2fb1fa Mon Sep 17 00:00:00 2001
From: Aditya Bharadwaj <adb@vt.edu>
Date: Wed, 3 May 2017 17:42:19 -0400
Subject: [PATCH 1/6] FIxed the browse button on upload page

---
 templates/upload_graph/index.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/templates/upload_graph/index.html b/templates/upload_graph/index.html
index b456d47f..a5cea909 100644
--- a/templates/upload_graph/index.html
+++ b/templates/upload_graph/index.html
@@ -58,7 +58,7 @@ <h1 class="text-center">Upload Graph</h1>
                                     <div class="input-group col-xs-12">
                                         <input type="text" class="browse form-control"
                                                placeholder="Required">
-                                        <span class="input-group-btn">
+                                        <span class="browse input-group-btn">
                                             <button class="browse btn btn-default" type="button">
                                                 <i class="glyphicon glyphicon-search"></i> Browse
                                             </button>
@@ -75,7 +75,7 @@ <h1 class="text-center">Upload Graph</h1>
                                     <div class="input-group col-xs-12">
                                         <input type="text" class="browse form-control"
                                                placeholder="Optional">
-                                        <span class="input-group-btn">
+                                        <span class="browse input-group-btn">
                                             <button class="browse btn btn-default" type="button">
                                                 <i class="glyphicon glyphicon-search"></i> Browse
                                             </button>

From 61e55187deb3986483c480fe0e620bfde27c6259 Mon Sep 17 00:00:00 2001
From: Aditya Bharadwaj <adbharadwaj@users.noreply.github.com>
Date: Mon, 15 May 2017 10:52:57 -0400
Subject: [PATCH 2/6] Update README.md

---
 README.md | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/README.md b/README.md
index 588b6f73..c5a64c20 100644
--- a/README.md
+++ b/README.md
@@ -78,14 +78,6 @@ WSGIScriptAlias / /path_to_GraphSpace/graphspace/wsgi.py
 
 Refer to https://docs.djangoproject.com/en/1.8/howto/deployment/wsgi/modwsgi/ if any problems occur with the setup.
 
-Testing the GraphSpace REST API
-=================================
-
-1. Follow the instructions above to run GraphSpace locally or on Apache2.
-2. Navigate to the tests directory in GraphSpace: `cd tests`
-2. Enter python command to run test cases: `python restapi_test.py`
-
-This script will test all the REST API commands supported by the GraphSpace server.  If something fails, it will display the error.
 
 Contributing
 =================

From ab00adf6bdbb4fc35a06c85f983471c5a2452dba Mon Sep 17 00:00:00 2001
From: Melvin Mathew <melvin15may@gmail.com>
Date: Fri, 19 May 2017 11:37:53 -0700
Subject: [PATCH 3/6] change the ID for layout exit modal

---
 static/js/graphs_page.js   | 2 +-
 templates/graph/index.html | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/static/js/graphs_page.js b/static/js/graphs_page.js
index 02da7544..5b621d0c 100644
--- a/static/js/graphs_page.js
+++ b/static/js/graphs_page.js
@@ -446,7 +446,7 @@ var graphPage = {
 
         $('#exitLayoutEditorBtn').click(function () {
             $('#exitLayoutBtn').removeClass('hidden');
-            $('#saveLayoutModal').modal('show');
+            $('#saveOnExitLayoutModal').modal('show');
         });
 
         $('#exitLayoutBtn').click(function () {
diff --git a/templates/graph/index.html b/templates/graph/index.html
index 8eb07ba2..9b0ace03 100644
--- a/templates/graph/index.html
+++ b/templates/graph/index.html
@@ -75,9 +75,9 @@ <h4 class="modal-title" id="myModalLabel">Share Graph</h4>
                 </div>
             </div>
         </div>
-
-        <!-- Save Layout Modal -->
-        <div class="modal fade" id="saveLayoutModal" tabindex="-1" role="dialog" aria-labelledby="saveLayoutModalLabel"
+    
+        <!-- Save on Exit Layout Modal -->
+        <div class="modal fade" id="saveOnExitLayoutModal" tabindex="-1" role="dialog" aria-labelledby="saveOnExitLayoutModalLabel"
              aria-hidden="true">
             <div class="modal-dialog">
                 <div class="modal-content">

From 2fc36d4a439faab7a588d9e631a5f74bc0dcc4a2 Mon Sep 17 00:00:00 2001
From: Melvin Mathew <melvin15may@gmail.com>
Date: Fri, 19 May 2017 12:55:41 -0700
Subject: [PATCH 4/6] different save button id for exit save button

---
 static/js/graphs_page.js   | 13 ++++---------
 templates/graph/index.html |  5 ++---
 2 files changed, 6 insertions(+), 12 deletions(-)

diff --git a/static/js/graphs_page.js b/static/js/graphs_page.js
index 5b621d0c..3428da88 100644
--- a/static/js/graphs_page.js
+++ b/static/js/graphs_page.js
@@ -430,14 +430,14 @@ var graphPage = {
 
         utils.initializeTabs();
 
-        $('#saveLayoutBtn').click(function () {
+        $('#saveOnExitLayoutBtn').click(function () {
             graphPage.cyGraph.contextMenus('get').destroy(); // Destroys the cytocscape context menu extension instance.
 
             cytoscapeGraph.showGraphInformation(graphPage.cyGraph);
             // display node data as a popup
             graphPage.cyGraph.unbind('tap').on('tap', graphPage.onTapGraphElement);
 
-            graphPage.saveLayout($('#layoutNameInput').val());
+            graphPage.saveLayout($('#saveOnExitLayoutNameInput').val(), '#saveOnExitLayoutModal');
         });
 
         $('#layoutEditorBtn').click(function () {
@@ -457,11 +457,6 @@ var graphPage = {
             graphPage.cyGraph.unbind('tap').on('tap', graphPage.onTapGraphElement);
         });
 
-        $('#saveLayoutModalBtn').click(function () {
-            $('#exitLayoutBtn').addClass('hidden');
-            $('#saveLayoutModal').modal('show');
-        });
-
         $('#ConfirmRemoveLayoutBtn').click(graphPage.layoutsTable.onConfirmRemoveGraph);
         $('#ConfirmUpdateLayoutBtn').click(graphPage.layoutsTable.onConfirmUpdateGraph);
 
@@ -584,7 +579,7 @@ var graphPage = {
         graphPage.cyGraph.layout(layout);
 
     },
-    saveLayout: function (layoutName) {
+    saveLayout: function (layoutName, modalNameId) {
         graphPage.cyGraph.elements().unselect();
 
         if (_.trim(layoutName).length === 0) {
@@ -636,7 +631,7 @@ var graphPage = {
                     }
                 },
                 successCallback = function (response) {
-                    $('#saveLayoutModal').modal('toggle');
+                    $(modalNameId).modal('toggle');
                     $('#PrivateLayoutsTable').bootstrapTable('refresh');
                     $('#SharedLayoutsTable').bootstrapTable('refresh');
                 },
diff --git a/templates/graph/index.html b/templates/graph/index.html
index 9b0ace03..be5b2809 100644
--- a/templates/graph/index.html
+++ b/templates/graph/index.html
@@ -89,14 +89,13 @@ <h4 class="modal-title" id="myModalLabel">Save Layout</h4>
                         <div class="layout_fields">
                             <div class="control-group">
                                 <label for="layoutNameInput">Layout Name</label>
-                                <input id="layoutNameInput" class="form-control" type="text" name="layout_name"
+                                <input id="saveOnExitLayoutNameInput" class="form-control" type="text" name="layout_name"
                                        autocomplete="off"/>
                             </div>
                             <br>
                             <div class="form-group">
-                                <button name="saveLayoutBtn" id="saveLayoutBtn"
+                                <button name="saveOnExitLayoutBtn" id="saveOnExitLayoutBtn"
                                         class="btn btn-success sidebar-nav-pills" data-target="#defaultSideBar">Save
-                                    Layout
                                 </button>
                                 <button type="button" id="exitLayoutBtn"
                                         class="btn btn-success hidden sidebar-nav-pills" data-target="#defaultSideBar"

From bd926dc7624040f8b28a8f97b96e9d6f0536f7b6 Mon Sep 17 00:00:00 2001
From: Melvin Mathew <melvin15may@gmail.com>
Date: Fri, 19 May 2017 13:24:38 -0700
Subject: [PATCH 5/6] added button and modal to save layouts without exiting
 the layout editor

---
 static/js/graphs_page.js                   | 15 +++++++++++
 templates/graph/index.html                 | 31 ++++++++++++++++++++++
 templates/graph/layout_editor_sidebar.html |  6 +++++
 3 files changed, 52 insertions(+)

diff --git a/static/js/graphs_page.js b/static/js/graphs_page.js
index 3428da88..559ecbf8 100644
--- a/static/js/graphs_page.js
+++ b/static/js/graphs_page.js
@@ -440,6 +440,16 @@ var graphPage = {
             graphPage.saveLayout($('#saveOnExitLayoutNameInput').val(), '#saveOnExitLayoutModal');
         });
 
+        $('#saveLayoutBtn').click(function () {
+            graphPage.cyGraph.contextMenus('get').destroy(); // Destroys the cytocscape context menu extension instance.
+
+            cytoscapeGraph.showGraphInformation(graphPage.cyGraph);
+            // display node data as a popup
+            graphPage.cyGraph.unbind('tap').on('tap', graphPage.onTapGraphElement);
+
+            graphPage.saveLayout($('#saveLayoutNameInput').val(), '#saveLayoutModal');
+        });
+
         $('#layoutEditorBtn').click(function () {
             graphPage.layoutEditor.init();
         });
@@ -449,6 +459,11 @@ var graphPage = {
             $('#saveOnExitLayoutModal').modal('show');
         });
 
+        $('#saveLayoutEditorBtn').click(function () {
+            $('#exitLayoutBtn').removeClass('hidden');
+            $('#saveLayoutModal').modal('show');
+        });
+
         $('#exitLayoutBtn').click(function () {
             graphPage.cyGraph.contextMenus('get').destroy(); // Destroys the cytocscape context menu extension instance.
 
diff --git a/templates/graph/index.html b/templates/graph/index.html
index be5b2809..b3e39dbc 100644
--- a/templates/graph/index.html
+++ b/templates/graph/index.html
@@ -110,6 +110,37 @@ <h4 class="modal-title" id="myModalLabel">Save Layout</h4>
             </div>
         </div>
 
+        <!-- Save Layout Modal -->
+        <div class="modal fade" id="saveLayoutModal" tabindex="-1" role="dialog" aria-labelledby="saveLayoutModalLabel"
+             aria-hidden="true">
+            <div class="modal-dialog">
+                <div class="modal-content">
+                    <div class="modal-header">
+                        <h4 class="modal-title" id="myModalLabel">Save Layout</h4>
+                    </div>
+                    <div class="modal-body">
+
+                        <div class="layout_fields">
+                            <div class="control-group">
+                                <label for="layoutNameInput">Layout Name</label>
+                                <input id="saveLayoutNameInput" class="form-control" type="text" name="layout_name"
+                                       autocomplete="off"/>
+                            </div>
+                            <br>
+                            <div class="form-group">
+                                <button name="saveLayoutBtn" id="saveLayoutBtn"
+                                        class="btn btn-success sidebar-nav-pills" data-target="#layoutEditorSideBar">Save
+                                    Layout
+                                </button>
+                                <button type="button" class="btn btn-danger" data-dismiss="modal">Close</button>
+                            </div>
+                        </div>
+
+                    </div>
+                </div>
+            </div>
+        </div>
+
     </div>
     <!--- Modals end here--->
     <div class="container-fluid zero-margin zero-padding">
diff --git a/templates/graph/layout_editor_sidebar.html b/templates/graph/layout_editor_sidebar.html
index 1a55bb0c..caa8742a 100644
--- a/templates/graph/layout_editor_sidebar.html
+++ b/templates/graph/layout_editor_sidebar.html
@@ -6,6 +6,12 @@
         <a id="exitLayoutEditorBtn" class="btn btn-xs d-inline-block" href="#">
             <i class="fa fa-chevron-left"></i> Exit Layout Editor
         </a>
+        <a id="saveLayoutEditorBtn" class="btn btn-xs d-inline-block" href="#">
+            Save
+        </a>
+    </li>
+
+    <li>
         <a id="startTourBtn" class="btn btn-xs d-inline-block" href="#">
             Start Tour
         </a>

From 2eef71d938a670da834ef8a198fc6a4374409073 Mon Sep 17 00:00:00 2001
From: Melvin Mathew <melvin15may@gmail.com>
Date: Fri, 19 May 2017 13:55:35 -0700
Subject: [PATCH 6/6] removed 'layout' from save button

---
 templates/graph/index.html | 1 -
 1 file changed, 1 deletion(-)

diff --git a/templates/graph/index.html b/templates/graph/index.html
index b3e39dbc..bea8db4b 100644
--- a/templates/graph/index.html
+++ b/templates/graph/index.html
@@ -130,7 +130,6 @@ <h4 class="modal-title" id="myModalLabel">Save Layout</h4>
                             <div class="form-group">
                                 <button name="saveLayoutBtn" id="saveLayoutBtn"
                                         class="btn btn-success sidebar-nav-pills" data-target="#layoutEditorSideBar">Save
-                                    Layout
                                 </button>
                                 <button type="button" class="btn btn-danger" data-dismiss="modal">Close</button>
                             </div>