Add alert dialog for (lockdown) options retrieval error

benburrill · Dec 10, 2017 · 03484d5 · 03484d5
1 parent 83bc2f9
commit 03484d5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/lockdown.html b/lockdown.html
@@ -75,6 +75,9 @@
 		</div>
 
 		<div id="alerts" style="display: none;">
+			<div id="alertRetrieveError" title="LeechBlock Lockdown">
+				<p><strong>Error:</strong> Unable to retrieve options from browser storage.</p>
+			</div>
 			<div id="alertNoDuration" title="LeechBlock Lockdown">
 				<p>Please enter a non-zero duration for the lockdown.</p>
 			</div>

diff --git a/lockdown.js b/lockdown.js
@@ -41,6 +41,7 @@ function initializeForm() {
 
 	function onError(error) {
 		warn("Cannot get options: " + error);
+		$("#alertRetrieveError").dialog("open");
 	}
 }
 
@@ -122,16 +123,15 @@ $("#activate").click(onActivate);
 $("#cancel").button();
 $("#cancel").click(onCancel);
 
-let alerts = ["alertNoDuration", "alertNoSets"];
-for (let alert of alerts) {
-	$(`#${alert}`).dialog({
-		autoOpen: false,
-		modal: true,
-		buttons: {
-			OK: function() { $(this).dialog("close"); }
-		}
-	});
-}
+// Initialize alert dialogs
+$("div[id^='alert']").dialog({
+	autoOpen: false,
+	modal: true,
+	width: 500,
+	buttons: {
+		OK: function () { $(this).dialog("close"); }
+	}
+});
 
 $("#form").show();