diff --git a/web/eslint.config.js b/web/eslint.config.js index 13b2fda7..daa4561a 100644 --- a/web/eslint.config.js +++ b/web/eslint.config.js @@ -23,7 +23,11 @@ export default defineConfig( languageOptions: { globals: { ...globals.browser, ...globals.node } }, rules: { // typescript owns undefined-name checks. - "no-undef": "off" + "no-undef": "off", + "@typescript-eslint/no-unused-vars": [ + "error", + { argsIgnorePattern: "^_", varsIgnorePattern: "^_", caughtErrorsIgnorePattern: "^_" } + ] } }, { diff --git a/web/package.json b/web/package.json index f49ab82d..2fa10002 100644 --- a/web/package.json +++ b/web/package.json @@ -16,6 +16,7 @@ "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch", "lint": "prettier --check . && eslint .", "format": "prettier --write .", + "format:changed": "git diff --name-only --relative --diff-filter=d HEAD^1 | xargs -r prettier --write --ignore-unknown", "test:unit": "vitest --project server", "test": "vitest --run --project server && playwright test", "test:storybook": "vitest --run --project storybook", diff --git a/web/src/app.html b/web/src/app.html index 2d7df91b..8e289155 100644 --- a/web/src/app.html +++ b/web/src/app.html @@ -21,7 +21,7 @@ %sveltekit.head% - +
%sveltekit.body%
diff --git a/web/src/routes/[handle]/[repo]/+layout@.svelte b/web/src/routes/[handle]/[repo]/+layout@.svelte index bbd51eb1..d6bafb90 100644 --- a/web/src/routes/[handle]/[repo]/+layout@.svelte +++ b/web/src/routes/[handle]/[repo]/+layout@.svelte @@ -18,9 +18,8 @@ : "overview"; }); - // the pull page runs a discussion rail down the full page height, so it places - // the repo header and tabs inside its own column instead of taking them here - const bare = $derived((page.route.id ?? "").startsWith("/[handle]/[repo]/pulls/[aturi]")); + // the pull page is full screen view + const bare = $derived((page.route.id ?? "").startsWith("/[handle]/[repo]/pulls/")); // commit and pull pages break out of the reading column, everything else // stays capped @@ -46,9 +45,9 @@ /> -
+
+ {#if !bare} - {/if} {@render children()}