Synced minima.js and .ts, MinimaCreateNotification parameter correcti…

…on with bgcolor
minima-global · May 26, 2021 · 3f393f0 · 3f393f0
1 parent 5b8f917
commit 3f393f0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/js/minima.js b/src/js/minima.js
@@ -646,7 +646,7 @@ function httpPostAsync(theUrl, params, callback){
 						try {
 							callback(JSON.parse(xmlHttp.responseText));
 						} catch (err) {
-							Minima.notify(err);
+							Minima.notify(err, '#BEB25B');
 						}
         	}
         }

diff --git a/src/minima.ts b/src/minima.ts
@@ -387,7 +387,7 @@ const Minima = {
 	/**
 	 * Notify the user with a Pop up message
 	 */
-	notify : function (message: string, bgcolor: string): void {
+	notify : function (message: string, bgcolor?: string): void {
 		//Log it..
 		Minima.log("Notify : "+message);
 
@@ -807,7 +807,11 @@ function httpPostAsync(theUrl: string, params: string, callback: Callback): void
 
 				//Send it to the callback function..
 				if (callback) {
-					callback(JSON.parse(xmlHttp.responseText));
+					try {
+						callback(JSON.parse(xmlHttp.responseText));
+					} catch (err) {
+						Minima.notify(err, '#BEB25B');
+					}
 				}
 			}
 	}
@@ -849,7 +853,7 @@ function httpGetAsync(theUrl: string, callback: (jsonresp: any) => void, logenab
 /**
  * Notifications
  */
-function MinimaCreateNotification(text: string, bgcolor: string = "#bbbbbb"): void {
+function MinimaCreateNotification(text: string, bgcolor?: string): void {
 	//First add the total overlay div
 	const notifydiv = document.createElement('div');