fix(sites): fix REPL styles (#8770)

* Push * Push fixes * Delete vite.config.js.timestamp-1687278902174-a59162675b2f6.mjs * Remove log * bump site-ki
2 years ago · fe60894066
parent 3858321b32
commit fe60894066
6 changed files with 17 additions and 11 deletions
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@ -182,8 +182,8 @@ importers:
        specifier: ^1.20.4
        version: 1.20.4(svelte@packages+svelte)(vite@4.3.9)
      '@sveltejs/site-kit':
-        specifier: 6.0.0-next.6
-        version: 6.0.0-next.6(@sveltejs/kit@1.20.4)(svelte@packages+svelte)
+        specifier: 6.0.0-next.8
+        version: 6.0.0-next.8(@sveltejs/kit@1.20.4)(svelte@packages+svelte)
      '@sveltejs/vite-plugin-svelte':
        specifier: ^2.4.1
        version: 2.4.1(svelte@packages+svelte)(vite@4.3.9)
@ -1822,8 +1822,8 @@ packages:
      svelte-local-storage-store: 0.4.0(svelte@packages+svelte)
    dev: false

-  /@sveltejs/site-kit@6.0.0-next.6(@sveltejs/kit@1.20.4)(svelte@packages+svelte):
-    resolution: {integrity: sha512-5+NjcAzksiCkQrgKe9Kj3L8yMqGSF73QJnV949gFOzxY2VjTNeCJCQ4fl7Ek64/j5YrLEdFaz0DbYNqZzes8qg==}
+  /@sveltejs/site-kit@6.0.0-next.8(@sveltejs/kit@1.20.4)(svelte@packages+svelte):
+    resolution: {integrity: sha512-3sCRKQnjhxJRR1FYpBBsGRqiV1cqSYFGAi/lJb4rt1U2+cvpRxJgJUHzrQqbfB4JfWpD63FcLKYU1+P+maaX/w==}
    peerDependencies:
      '@sveltejs/kit': ^1.0.0
      svelte: ^3.54.0 || ^4.0.0-next.0 || ^4.0.0
--- a/sites/svelte.dev/package.json
+++ b/sites/svelte.dev/package.json
@ -30,7 +30,7 @@
 		"@resvg/resvg-js": "^2.4.1",
 		"@sveltejs/adapter-vercel": "^3.0.1",
 		"@sveltejs/kit": "^1.20.4",
-		"@sveltejs/site-kit": "6.0.0-next.6",
+		"@sveltejs/site-kit": "6.0.0-next.8",
 		"@sveltejs/vite-plugin-svelte": "^2.4.1",
 		"@types/marked": "^5.0.0",
 		"@types/node": "^20.3.1",
--- a/sites/svelte.dev/src/app.html
+++ b/sites/svelte.dev/src/app.html
@ -30,6 +30,6 @@
 		%sveltekit.head%
 	</head>
 	<body data-sveltekit-preload-code="hover">
-		<div>%sveltekit.body%</div>
+		<div style="height: 100%">%sveltekit.body%</div>
 	</body>
 </html>
--- a/sites/svelte.dev/src/routes/(authed)/repl/[id]/+page.server.js
+++ b/sites/svelte.dev/src/routes/(authed)/repl/[id]/+page.server.js
@ -11,6 +11,6 @@ export async function load({ fetch, params, url }) {

 	return {
 		gist,
-		version: url.searchParams.get('version') || '3'
+		version: url.searchParams.get('version') || '4.0.0-next.2'
 	};
 }
--- a/sites/svelte.dev/src/routes/(authed)/repl/[id]/+page.svelte
+++ b/sites/svelte.dev/src/routes/(authed)/repl/[id]/+page.svelte
@ -32,7 +32,7 @@

 	onMount(() => {
 		if (data.version !== 'local') {
-			fetch(`https://unpkg.com/svelte@${data.version || '3'}/package.json`)
+			fetch(`https://unpkg.com/svelte@${data.version || '4.0.0-next.2'}/package.json`)
 				.then((r) => r.json())
 				.then((pkg) => {
 					version = pkg.version;
@ -101,7 +101,8 @@
 <style>
 	.repl-outer {
 		position: relative;
-		height: calc(100vh - var(--sk-nav-height));
+		height: calc(100% - var(--sk-nav-height));
+		height: calc(100dvh - var(--sk-nav-height));
 		--app-controls-h: 5.6rem;
 		--pane-controls-h: 4.2rem;
 		overflow: hidden;
--- a/sites/svelte.dev/src/routes/tutorial/[slug]/+page.svelte
+++ b/sites/svelte.dev/src/routes/tutorial/[slug]/+page.svelte
@ -67,15 +67,20 @@
 	$: mobile = width < 768;

 	function reset() {
-		repl.update({
+		repl.set({
 			files: data.tutorial.initial.map(clone)
 		});
+
+		//! BUG: Fix handleChange on REPL side, setting repl.set doesn't trigger it, and repl.update doesn't even work
+		completed = false;
 	}

 	function complete() {
-		repl.update({
+		repl.set({
 			files: data.tutorial.complete.map(clone)
 		});
+
+		completed = true;
 	}

 	let completed = false;