Skip to content

Commit 47bd40f

Browse files
fix(web): always-visible wordmark in navbar
The header used <img src="/img/logo.png"> followed by a "Code for Philly" wordmark gated with `hidden md:inline`. The image asset has never existed in the repo, so it rendered as an invisible broken-image on every viewport. Desktop nobody noticed because the wordmark text filled the spot; mobile (where hidden md:inline hides the wordmark) showed an empty home link with no tappable visual. Drop the broken <img> and unhide the wordmark at all breakpoints. The home link is now always visible, always labeled, always tappable. The actual logo asset is a future TODO; the wordmark covers the functional need. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent b877e6e commit 47bd40f

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

apps/web/src/components/AppHeader.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -187,16 +187,16 @@ export function AppHeader() {
187187
return (
188188
<header className="sticky top-0 z-40 w-full border-b border-border bg-background/95 backdrop-blur supports-[backdrop-filter]:bg-background/60 shadow-sm print:hidden">
189189
<div className="container mx-auto flex h-14 items-center px-4 gap-4">
190-
{/* Logo */}
190+
{/* Wordmark — always visible. The image asset is a future TODO,
191+
but the wordmark alone gives the home link a tappable target
192+
on every viewport, including mobile where the desktop nav
193+
isn't present. */}
191194
<Link
192195
to="/"
193-
className="flex items-center gap-2 shrink-0"
196+
className="flex items-center gap-2 shrink-0 font-semibold text-foreground"
194197
aria-label="Code for Philly home"
195198
>
196-
<img src="/img/logo.png" alt="" height={32} className="h-8 w-auto" />
197-
<span className="hidden md:inline font-semibold text-foreground">
198-
Code for Philly
199-
</span>
199+
Code for Philly
200200
</Link>
201201

202202
{/* Desktop nav */}

0 commit comments

Comments
 (0)