From ca981663112a2c6bf0c4b05d7aeaf4bb1cbd606b Mon Sep 17 00:00:00 2001 From: Luke Bowsher Date: Sun, 22 Mar 2026 14:07:20 -0400 Subject: [PATCH] fix: deduplicate nav bars, centralize layout, and clean up UI inconsistencies MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove duplicate nav (ThemeToggle + AuthButtonServer) from team page, replace with breadcrumb - Remove duplicate "Home" header from new-league page - Move min-h-screen bg-background to body in layout.tsx, remove from 10 pages - Update loading skeletons to match current page structure (no fake nav placeholders) - Replace raw input/button in search-component with shadcn components + Enter key support - Fix getSession() → getUser() in one-tap-component.tsx - Replace all text-liquid-lava with semantic tokens (text-primary, text-accent, etc.) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- app/draft/[id]/draft-info-panel.tsx | 2 +- app/draft/[id]/draft-queue.tsx | 2 +- app/draft/[id]/error.tsx | 2 +- app/draft/[id]/loading.tsx | 74 ++++++++----------- app/draft/[id]/page.tsx | 28 +++---- app/error.tsx | 2 +- app/layout.tsx | 2 +- app/league/[id]/error.tsx | 2 +- app/league/[id]/league-home.tsx | 4 +- app/league/[id]/loading.tsx | 43 ++++------- app/league/[id]/page.tsx | 44 +++++------ app/league/[id]/team/[teamid]/page.tsx | 50 ++++--------- .../[id]/team/[teamid]/search-component.tsx | 18 +++-- app/login/loading.tsx | 4 +- app/login/one-tap-component.tsx | 6 +- app/new-league/league-creator.tsx | 18 ++--- app/rankings/ncaa/rankings-page.tsx | 4 +- 17 files changed, 122 insertions(+), 183 deletions(-) diff --git a/app/draft/[id]/draft-info-panel.tsx b/app/draft/[id]/draft-info-panel.tsx index 3ff2f05..d257109 100644 --- a/app/draft/[id]/draft-info-panel.tsx +++ b/app/draft/[id]/draft-info-panel.tsx @@ -214,7 +214,7 @@ export default function DraftInfoPanel({ checked={autoPickEnabled} onChange={onToggleAutoPick} disabled={isPending} - className="h-4 w-4 text-liquid-lava rounded focus:ring-liquid-lava" + className="h-4 w-4 accent-primary rounded" /> diff --git a/app/draft/[id]/draft-queue.tsx b/app/draft/[id]/draft-queue.tsx index 8aec142..17e4d76 100644 --- a/app/draft/[id]/draft-queue.tsx +++ b/app/draft/[id]/draft-queue.tsx @@ -227,7 +227,7 @@ export default function DraftQueue({ teamId, draftId }: DraftQueueProps) { + ); }; -export default SearchComponent; \ No newline at end of file +export default SearchComponent; diff --git a/app/login/loading.tsx b/app/login/loading.tsx index 1b78977..ca35d1c 100644 --- a/app/login/loading.tsx +++ b/app/login/loading.tsx @@ -1,6 +1,6 @@ export default function LoginLoading() { return ( -
+
{/* Logo skeleton */} @@ -10,7 +10,7 @@ export default function LoginLoading() {
-
+
{/* Divider skeleton */} diff --git a/app/login/one-tap-component.tsx b/app/login/one-tap-component.tsx index 1a8f553..80879d8 100644 --- a/app/login/one-tap-component.tsx +++ b/app/login/one-tap-component.tsx @@ -29,11 +29,11 @@ const OneTapComponent = ({ redirectPath }: { redirectPath?: string }) => { console.log('Nonce: ', nonce, hashedNonce) // check if there's already an existing session before initializing the one-tap UI - const { data, error } = await supabase.auth.getSession() + const { data: { user }, error } = await supabase.auth.getUser() if (error) { - console.error('Error getting session', error) + console.error('Error getting user', error) } - if (data.session) { + if (user) { router.push(redirectPath || '/') return } diff --git a/app/new-league/league-creator.tsx b/app/new-league/league-creator.tsx index b2958db..0c6b702 100644 --- a/app/new-league/league-creator.tsx +++ b/app/new-league/league-creator.tsx @@ -1,5 +1,4 @@ "use client"; -import Link from 'next/link'; import { useState } from 'react'; import { addLeague } from './league-action'; import { Button } from "@/components/ui/button"; @@ -10,14 +9,8 @@ export default function LeagueCreator() { const [draftEnabled, setDraftEnabled] = useState(true); return ( -
-
- - Home - -
-
-
+
+
Create a New League @@ -96,14 +89,14 @@ export default function LeagueCreator() {
{/* Draft Settings */} -
+
setDraftEnabled(e.target.checked)} - className="h-4 w-4 text-liquid-lava rounded focus:ring-liquid-lava" + className="h-4 w-4 accent-primary rounded" />
); } diff --git a/app/rankings/ncaa/rankings-page.tsx b/app/rankings/ncaa/rankings-page.tsx index 3f045de..17ecb00 100644 --- a/app/rankings/ncaa/rankings-page.tsx +++ b/app/rankings/ncaa/rankings-page.tsx @@ -237,8 +237,7 @@ export default function RankingsPage({ }, []); return ( -
-
+

NCAA Rankings

@@ -370,7 +369,6 @@ export default function RankingsPage({ /> -

); }