From 7266f6144161144a5f5967b03bd564b51e6c21ce Mon Sep 17 00:00:00 2001
From: Sonia Zorba <zorba@oats.inaf.it>
Date: Wed, 14 Dec 2016 13:57:14 +0100
Subject: [PATCH] Improved loading animation

---
 .../src/main/webapp/WEB-INF/templates/master.xhtml        | 7 +++++++
 .../src/main/webapp/consistencyChecks.xhtml               | 2 +-
 .../src/main/webapp/resources/js/edit-tapschema.js        | 4 ++--
 .../src/main/webapp/resources/js/script.js                | 7 +++++++
 .../src/main/webapp/schemaSelection.xhtml                 | 4 ++--
 .../src/main/webapp/tapSchemaEditing.xhtml                | 8 +-------
 6 files changed, 20 insertions(+), 12 deletions(-)
 create mode 100644 TapSchemaManagerWebApp/src/main/webapp/resources/js/script.js

diff --git a/TapSchemaManagerWebApp/src/main/webapp/WEB-INF/templates/master.xhtml b/TapSchemaManagerWebApp/src/main/webapp/WEB-INF/templates/master.xhtml
index 196c000..21b0916 100644
--- a/TapSchemaManagerWebApp/src/main/webapp/WEB-INF/templates/master.xhtml
+++ b/TapSchemaManagerWebApp/src/main/webapp/WEB-INF/templates/master.xhtml
@@ -18,10 +18,17 @@
         <h:outputStylesheet library="css" name="style.css"></h:outputStylesheet>
         <h:outputScript library="js" name="lib/jquery-1.11.3.min.js"></h:outputScript>
         <h:outputScript library="js" name="lib/bootstrap.min.js"></h:outputScript>
+        <h:outputScript library="js" name="script.js"></h:outputScript>
         <ui:insert name="scripts"></ui:insert>
     </h:head>
 
     <h:body>
         <ui:insert name="content" />
+
+        <div class="loading hide">
+            <div class="icon-wrapper">
+                <span class="glyphicon glyphicon-refresh animate-spin"></span>
+            </div>
+        </div>
     </h:body>
 </html>
\ No newline at end of file
diff --git a/TapSchemaManagerWebApp/src/main/webapp/consistencyChecks.xhtml b/TapSchemaManagerWebApp/src/main/webapp/consistencyChecks.xhtml
index ec7ef80..5fa2503 100644
--- a/TapSchemaManagerWebApp/src/main/webapp/consistencyChecks.xhtml
+++ b/TapSchemaManagerWebApp/src/main/webapp/consistencyChecks.xhtml
@@ -82,7 +82,7 @@
                                 Back
                             </h:commandLink>
                             &#160;
-                            <h:commandLink action="#{consistency.proceed()}" class="btn btn-primary">
+                            <h:commandLink action="#{consistency.proceed()}" class="btn btn-primary" onclick="showWaiting()">
                                 Proceed
                                 <span class="glyphicon glyphicon-arrow-right"></span>
                             </h:commandLink>
diff --git a/TapSchemaManagerWebApp/src/main/webapp/resources/js/edit-tapschema.js b/TapSchemaManagerWebApp/src/main/webapp/resources/js/edit-tapschema.js
index 99ab01d..5d09fcd 100644
--- a/TapSchemaManagerWebApp/src/main/webapp/resources/js/edit-tapschema.js
+++ b/TapSchemaManagerWebApp/src/main/webapp/resources/js/edit-tapschema.js
@@ -162,10 +162,10 @@
             }
             switch (data.status) {
                 case "begin":
-                    $('.loading').removeClass('hide');
+                    showWaiting();
                     break;
                 case "complete":
-                    $('.loading').addClass('hide');
+                    hideWaiting();
                     break;
             }
         });
diff --git a/TapSchemaManagerWebApp/src/main/webapp/resources/js/script.js b/TapSchemaManagerWebApp/src/main/webapp/resources/js/script.js
new file mode 100644
index 0000000..a20c430
--- /dev/null
+++ b/TapSchemaManagerWebApp/src/main/webapp/resources/js/script.js
@@ -0,0 +1,7 @@
+
+window.showWaiting = function () {
+    $('.loading').removeClass('hide');
+};
+window.hideWaiting = function () {
+    $('.loading').addClass('hide');
+};
diff --git a/TapSchemaManagerWebApp/src/main/webapp/schemaSelection.xhtml b/TapSchemaManagerWebApp/src/main/webapp/schemaSelection.xhtml
index a7f422f..01bd503 100644
--- a/TapSchemaManagerWebApp/src/main/webapp/schemaSelection.xhtml
+++ b/TapSchemaManagerWebApp/src/main/webapp/schemaSelection.xhtml
@@ -54,7 +54,7 @@
                                         </p>
                                     </div>
                                     <div class="form-group">
-                                        <h:commandButton value="Edit Tapschema" class="btn btn-primary" action="#{schemaSelection.edit()}"></h:commandButton>
+                                        <h:commandButton value="Edit Tapschema" class="btn btn-primary" action="#{schemaSelection.edit()}" onclick="showWaiting()"></h:commandButton>
                                     </div>
                                 </div>
                             </h:panelGroup>
@@ -78,7 +78,7 @@
                                     </div>
 
                                     <div class="form-group">
-                                        <h:commandButton value="Create TAP_SCHEMA" class="btn btn-primary" action="#{schemaSelection.create()}"></h:commandButton>
+                                        <h:commandButton value="Create TAP_SCHEMA" class="btn btn-primary" action="#{schemaSelection.create()}" onclick="showWaiting()"></h:commandButton>
                                     </div>
                                 </div>
                             </h:panelGroup>
diff --git a/TapSchemaManagerWebApp/src/main/webapp/tapSchemaEditing.xhtml b/TapSchemaManagerWebApp/src/main/webapp/tapSchemaEditing.xhtml
index 8440fa0..113af63 100644
--- a/TapSchemaManagerWebApp/src/main/webapp/tapSchemaEditing.xhtml
+++ b/TapSchemaManagerWebApp/src/main/webapp/tapSchemaEditing.xhtml
@@ -35,7 +35,7 @@
                     <f:ajax execute="@form" render="@form" onevent="TSM.notifyUpdate"/>
                 </h:commandLink>
                 &#160;
-                <h:commandLink class="btn btn-info" action="#{tapSchemaEditing.reload()}">
+                <h:commandLink class="btn btn-info" action="#{tapSchemaEditing.reload()}" onclick="showWaiting()">
                     <span class="glyphicon glyphicon-refresh"></span>
                     Reload all
                 </h:commandLink>
@@ -788,11 +788,5 @@
                 </div>
             </div>
         </div>
-
-        <div class="loading hide">
-            <div class="icon-wrapper">
-                <span class="glyphicon glyphicon-refresh animate-spin"></span>
-            </div>
-        </div>
     </ui:define>
 </ui:composition>
\ No newline at end of file
-- 
GitLab