fix(login): remove py-6 padding from layout on auth pages to prevent scrolling
Some checks failed
CI / Unit & Component Tests (pull_request) Successful in 2m25s
CI / Backend Unit Tests (pull_request) Successful in 2m10s
CI / E2E Tests (pull_request) Failing after 29m35s
CI / Unit & Component Tests (push) Successful in 2m27s
CI / Backend Unit Tests (push) Successful in 2m17s
CI / E2E Tests (push) Failing after 30m42s
Some checks failed
CI / Unit & Component Tests (pull_request) Successful in 2m25s
CI / Backend Unit Tests (pull_request) Successful in 2m10s
CI / E2E Tests (pull_request) Failing after 29m35s
CI / Unit & Component Tests (push) Successful in 2m27s
CI / Backend Unit Tests (push) Successful in 2m17s
CI / E2E Tests (push) Failing after 30m42s
The global layout wrapped all pages in <main class="py-6">, adding 48px of vertical padding. Combined with min-h-screen on the login page div, the total height exceeded 100vh and made the page scrollable. Auth pages (/login, /forgot-password, /reset-password) now get no padding from the layout — the same path check already used to hide the nav header. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit was merged in pull request #65.
This commit is contained in:
@@ -24,6 +24,10 @@ onMount(() => {
|
|||||||
hydrated = true;
|
hydrated = true;
|
||||||
});
|
});
|
||||||
|
|
||||||
|
const isAuthPage = $derived(
|
||||||
|
['/login', '/forgot-password', '/reset-password'].some((p) => page.url.pathname.startsWith(p))
|
||||||
|
);
|
||||||
|
|
||||||
let userMenuOpen = $state(false);
|
let userMenuOpen = $state(false);
|
||||||
|
|
||||||
const userInitials = $derived.by(() => {
|
const userInitials = $derived.by(() => {
|
||||||
@@ -47,7 +51,7 @@ function clickOutside(node: HTMLElement) {
|
|||||||
</script>
|
</script>
|
||||||
|
|
||||||
<div class="min-h-screen bg-canvas" data-hydrated={hydrated || undefined}>
|
<div class="min-h-screen bg-canvas" data-hydrated={hydrated || undefined}>
|
||||||
{#if !['/login', '/forgot-password', '/reset-password'].some((p) => page.url.pathname.startsWith(p))}
|
{#if !isAuthPage}
|
||||||
<header class="sticky top-0 z-50 border-b border-line-2 bg-surface">
|
<header class="sticky top-0 z-50 border-b border-line-2 bg-surface">
|
||||||
<!-- De Gruyter Brill purple accent strip -->
|
<!-- De Gruyter Brill purple accent strip -->
|
||||||
<div class="h-1 bg-brand-purple"></div>
|
<div class="h-1 bg-brand-purple"></div>
|
||||||
@@ -194,7 +198,7 @@ function clickOutside(node: HTMLElement) {
|
|||||||
</header>
|
</header>
|
||||||
{/if}
|
{/if}
|
||||||
|
|
||||||
<main class="py-6">
|
<main class={isAuthPage ? '' : 'py-6'}>
|
||||||
{@render children()}
|
{@render children()}
|
||||||
</main>
|
</main>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Reference in New Issue
Block a user