From 8dc9d4c083b2dc1a827e81df2e26242fc16740ef Mon Sep 17 00:00:00 2001
From: Johannes Zellner <johannes@cloudron.io>
Date: Sat, 28 Jan 2017 14:49:07 -0800
Subject: [PATCH] webadmin: Give better feedback on update schedule saving

---
 webadmin/src/views/settings.html | 5 +++--
 webadmin/src/views/settings.js   | 3 +++
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/webadmin/src/views/settings.html b/webadmin/src/views/settings.html
index 09809644c..23bfe2129 100644
--- a/webadmin/src/views/settings.html
+++ b/webadmin/src/views/settings.html
@@ -394,7 +394,7 @@
         <div class="col-md-12">
             Configure the auto-update schedule for the platform and for the installed apps:<br/>
             <br/>
-            <p class="text-danger text-center" ng-show="autoUpdate.error">{{ autoUpdate.error }}</p>
+            <p class="text-danger text-center text-bold" ng-show="autoUpdate.error">{{ autoUpdate.error }}</p>
         </div>
     </div>
 
@@ -423,7 +423,8 @@
 
     <div class="row">
         <div class="col-md-12">
-            <button class="btn btn-outline btn-primary pull-right" ng-click="autoUpdate.submit()" ng-disabled="autoUpdate.busy || autoUpdate.pattern === autoUpdate.currentPattern"><i class="fa fa-circle-o-notch fa-spin" ng-show="autoUpdate.busy"></i> Save</button>
+            <p class="text-success pull-right text-bold" ng-show="autoUpdate.success && autoUpdate.pattern === autoUpdate.currentPattern">Saved</p>
+            <button class="btn btn-outline btn-primary pull-right" ng-hide="autoUpdate.success && autoUpdate.pattern === autoUpdate.currentPattern" ng-click="autoUpdate.submit()" ng-disabled="autoUpdate.busy || autoUpdate.pattern === autoUpdate.currentPattern"><i class="fa fa-circle-o-notch fa-spin" ng-show="autoUpdate.busy"></i> Save</button>
         </div>
     </div>
 </div>
diff --git a/webadmin/src/views/settings.js b/webadmin/src/views/settings.js
index 93bb6a51c..ac1810ccd 100644
--- a/webadmin/src/views/settings.js
+++ b/webadmin/src/views/settings.js
@@ -422,6 +422,7 @@ angular.module('Application').controller('SettingsController', ['$scope', '$loca
 
     $scope.autoUpdate = {
         busy: false,
+        success: false,
         error: '',
         pattern: '',
         currentPattern: '',
@@ -431,12 +432,14 @@ angular.module('Application').controller('SettingsController', ['$scope', '$loca
 
             $scope.autoUpdate.error = '';
             $scope.autoUpdate.busy = true;
+            $scope.autoUpdate.success = false;
 
             Client.setAutoupdatePattern($scope.autoUpdate.pattern, function (error) {
                 if (error) $scope.autoUpdate.error = error.message;
                 else $scope.autoUpdate.currentPattern = $scope.autoUpdate.pattern;
 
                 $scope.autoUpdate.busy = false;
+                $scope.autoUpdate.success = true;
             });
         }
     };