[fix] signal that pings pause prio to alert opening

dummdidumm · dummdidumm · commit 47178568edb9 · 2022-12-13T16:45:15.000+01:00
Fixes sveltejs#126
diff --git a/content/tutorial/common/src/__client.js b/content/tutorial/common/src/__client.js
@@ -1,3 +1,19 @@
+// Hack into the alert that's used in some tutorials and send a message prior to the alert,
+// else the parent thinks we lost contact and wrongfully reloads the page.
+// The drawback is that alert is no longer blocking, but no tutorial relies on this.
+const alert = window.alert;
+window.alert = (message) => {
+	parent.postMessage(
+		{
+			type: 'ping-pause'
+		},
+		'*'
+	);
+	setTimeout(() => {
+		alert(message);
+	});
+};
+
 window.addEventListener('message', async (e) => {
 	if (e.data.type === 'fetch') {
 		const names = e.data.names;
@@ -59,7 +75,7 @@ function ping() {
 	);
 }
 
-setInterval(ping, 50);
+setInterval(ping, 100);
 ping();
 
 if (import.meta.hot) {
diff --git a/src/routes/tutorial/[slug]/+page.svelte b/src/routes/tutorial/[slug]/+page.svelte
@@ -254,7 +254,7 @@
 		if (e.origin !== adapter.base) return;
 
 		if (e.data.type === 'ping') {
-			path = e.data.data.path;
+			path = e.data.data.path ?? path;
 
 			clearTimeout(timeout);
 			timeout = setTimeout(() => {
@@ -264,7 +264,9 @@
 				loading = true;
 				set_iframe_src(adapter.base + path);
 				loading = false;
-			}, 500);
+			}, 1000);
+		} else if (e.data.type === 'ping-pause') {
+			clearTimeout(timeout);
 		}
 	}
 
