Oliver/dow-prod-tracker
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix missed .mjs db:seed script + Dow navy bg behind logo

Browse source 
Two smalls:

1) package.json db:seed still said seed-dow.cjs from the earlier
   CJS→ESM rename — file bump missed it because I hadn't read the file
   in that tool pass. With the Dockerfile now producing .mjs the seed
   would have failed again for a different, dumber reason. Fixed.

2) Dow Jones wordmark is white on transparent, so it vanished into the
   sidebar's light background. Added bg-[#002B5C] (Dow Jones brand
   navy) to the logo header in both the desktop sidebar and the mobile
   sheet. Now the logo actually reads.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
DJP 2026-04-21 08:56:53 -04:00
parent 29e76b7e33
commit e439ea09db
2 changed files with 5 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
package.json
View file

@ -12,7 +12,7 @@
"db:generate": "prisma generate",
"db:migrate": "prisma migrate dev",
"db:push": "prisma db push",
"db:seed": "node prisma/seed-dow.cjs",
"db:seed": "node prisma/seed-dow.mjs",
"db:seed-legacy": "tsx prisma/seed.ts",
"db:studio": "prisma studio",
"db:seed-tracker": "tsx prisma/seed-tracker-data.ts",

11
src/components/layout/sidebar.tsx
View file

@ -171,17 +171,14 @@ export function Sidebar() {
role="navigation"
aria-label="Sidebar"
>
{/* Logo / wordmark */}
{/* Logo / wordmark — Dow Jones logo is white, needs a dark
background to read. #002B5C is Dow Jones's brand navy. */}
<div className={cn(
"flex h-14 items-center border-b",
"flex h-14 items-center border-b bg-[#002B5C]",
isCollapsed ? "justify-center px-2" : "px-4"
)}>
{!isCollapsed && (
<Link href="/dashboard" className="flex items-center" aria-label="Dow Jones Studio Tracker">
{/* Plain <img> (not next/image) — basePath-handling in next/image
is fiddly and the file is ~4KB so image optimisation adds
nothing. Skipping Next's image pipeline avoids the whole
`/_next/image?url=...` + domain-allowlist dance. */}
{/* eslint-disable-next-line @next/next/no-img-element */}
<img
src={`${process.env.NEXT_PUBLIC_BASE_PATH ?? ""}/navbar-logo.png`}
@ -235,7 +232,7 @@ export function MobileSidebar() {
<Sheet open={isMobileOpen} onOpenChange={setMobileOpen}>
<SheetContent side="left" className="w-60 p-0">
<SheetTitle className="sr-only">Navigation</SheetTitle>
<div className="flex h-14 items-center border-b px-4">
<div className="flex h-14 items-center border-b bg-[#002B5C] px-4">
{/* eslint-disable-next-line @next/next/no-img-element */}
<img
src={`${process.env.NEXT_PUBLIC_BASE_PATH ?? ""}/navbar-logo.png`}