Remove set_iframe_src and loaded to prevent app-hiding during reload

Author: brandonmcconnell

Diff for: src/routes/tutorial/[slug]/+page.svelte

@@ -107,18 +107,13 @@
 	}
 
 	onMount(() => {
-		function on_iframe_load() {
-			iframe.classList.add('loaded');
-		}
 		function destroy() {
-			iframe.removeEventListener('load', on_iframe_load);
 			if (adapter) {
 				adapter.destroy();
 			}
 		}
 
 		document.addEventListener('pagehide', destroy);
-		iframe.addEventListener('load', on_iframe_load);
 		return destroy;
 	});
 
@@ -143,7 +138,7 @@
 
 			if (adapter && path !== data.exercise.path) {
 				path = data.exercise.path;
-				set_iframe_src(adapter.base + path);
+				iframe.src = adapter.base + path;
 			}
 
 			loading = false;
@@ -171,7 +166,7 @@
 				status = s;
 			});
 
-			set_iframe_src(adapter.base + path);
+			iframe.src = adapter.base + path;
 		}
 
 		await new Promise((fulfil, reject) => {
@@ -190,7 +185,7 @@
 				if (!called && adapter) {
 					// Updating the iframe too soon sometimes results in a blank screen,
 					// so we try again after a short delay if we haven't heard back
-					set_iframe_src(adapter.base + path);
+					iframe.src = adapter.base + path;
 				}
 			}, 5000);
 
@@ -203,7 +198,7 @@
 
 		if (reload_iframe) {
 			await new Promise((fulfil) => setTimeout(fulfil, 200));
-			set_iframe_src(adapter.base + path);
+			iframe.src = adapter.base + path;
 		}
 
 		return adapter;
@@ -230,7 +225,7 @@
 		clearTimeout(reload_timeout);
 		reload_timeout = setTimeout(() => {
 			if (adapter) {
-				set_iframe_src(adapter.base + path);
+				iframe.src = adapter.base + path;
 			}
 		}, 1000);
 	}
@@ -287,7 +282,7 @@
 
 				// we lost contact, refresh the page
 				loading = true;
-				set_iframe_src(adapter.base + path);
+				iframe.src = adapter.base + path;
 				loading = false;
 			}, 1000);
 		} else if (e.data.type === 'ping-pause') {
@@ -316,24 +311,12 @@
 		return code.replace(/\s+/g, ' ').trim();
 	}
 
-	/** @param {string} src */
-	function set_iframe_src(src) {
-		// removing the iframe from the document allows us to
-		// change the src without adding a history entry, which
-		// would make back/forward traversal very annoying
-		const parentNode = /** @type {HTMLElement} */ (iframe.parentNode);
-		iframe.classList.remove('loaded');
-		parentNode?.removeChild(iframe);
-		iframe.src = src;
-		parentNode?.appendChild(iframe);
-	}
-
 	/** @param {string} path */
 	function route_to(path) {
 		if (adapter) {
 			const url = new URL(path, adapter.base);
 			path = url.pathname + url.search + url.hash;
-			set_iframe_src(adapter.base + path);
+			iframe.src = adapter.base + path;
 		}
 	}
 
@@ -468,7 +451,7 @@
 						{loading}
 						on:refresh={() => {
 							if (adapter) {
-								set_iframe_src(adapter.base + path);
+								iframe.src = adapter.base + path;
 							}
 						}}
 						on:change={(e) => nav_to(e.detail.value)}
@@ -556,10 +539,6 @@
 		background: var(--sk-back-2);
 	}
 
-	iframe:not(.loaded) {
-		display: none;
-	}
-
 	.editor-container {
 		position: relative;
 		background-color: var(--sk-back-3);