From dcbad4d54b627b75fb8a6e6c3693a33c9343d184 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <matthewrdillon@gmail.com>
Date: Tue, 12 Dec 2017 07:58:28 -0700
Subject: [PATCH] BUG: Editing collections loading indicator (#63)

---
 app/controllers/collections/create.js      | 8 ++++++--
 app/controllers/collections/detail/edit.js | 8 ++++++--
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/app/controllers/collections/create.js b/app/controllers/collections/create.js
index 8b27025..c927cc9 100644
--- a/app/controllers/collections/create.js
+++ b/app/controllers/collections/create.js
@@ -29,11 +29,15 @@ export default Controller.extend(ValidationMixin, {
   actions: {
     onSave(changeset) {
       const postSave = () => { this.transitionToRoute('collections.index'); };
-      return this.validationSave(changeset, postSave);
+      return this.transitionToRoute('loading').then(() => {
+        return this.validationSave(changeset, postSave);
+      });
     },
     onCancel(changeset) {
       const postCancel = () => { this.transitionToRoute('collections.index'); };
-      return this.validationCancel(changeset, postCancel);
+      return this.transitionToRoute('loading').then(() => {
+        return this.validationCancel(changeset, postCancel);
+      });
     },
   },
 });
diff --git a/app/controllers/collections/detail/edit.js b/app/controllers/collections/detail/edit.js
index 205a4aa..978331a 100644
--- a/app/controllers/collections/detail/edit.js
+++ b/app/controllers/collections/detail/edit.js
@@ -32,14 +32,18 @@ export default Controller.extend(ValidationMixin, {
         // Use the model's ID here because of the ArrayProxy in the route
         this.transitionToRoute('collections.detail', this.get('model.id'));
       };
-      return this.validationSave(changesets, postSave);
+      return this.transitionToRoute('loading').then(() => {
+        return this.validationSave(changesets, postSave);
+      });
     },
     onCancel(changesets) {
       const postCancel = () => {
         // Use the model's ID here because of the ArrayProxy in the route
         return this.transitionToRoute('collections.detail', this.get('model.id'));
       };
-      return this.validationCancel(changesets, postCancel);
+      return this.transitionToRoute('loading').then(() => {
+        return this.validationCancel(changesets, postCancel);
+      });
     },
   },
 });