Skip to content

Commit 8000526

Browse files
committed
fix: use t.bgPrimary instead of t.bg in Dashboard.tsx to fix Netlify build
1 parent d8c88bf commit 8000526

4 files changed

Lines changed: 13 additions & 2 deletions

File tree

changelog.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ Format based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
88

99
### Fixed
1010

11+
- Fixed Netlify build error: `t.bg` property does not exist on theme classes
12+
- Changed `t.bg` to `t.bgPrimary` in Dashboard.tsx session detail container (line 1395)
13+
- Theme classes use `bgPrimary` not `bg` for primary background color
14+
1115
- Fixed mobile scrolling in session detail view (fixes #13)
1216
- Added `overscroll-contain`, `touch-pan-y`, and `WebkitOverflowScrolling: 'touch'` to messages container
1317
- Session detail uses absolute positioning on mobile with background color to cover session list

files.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ React frontend application.
8888
| ----------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
8989
| `auth.tsx` | Auth wrapper for WorkOS AuthKit with useAuth hook, session rehydration, and token refresh handling |
9090
| `utils.ts` | Utility functions (cn for classnames) |
91-
| `theme.tsx` | Theme context with page-specific defaults (PAGE_THEME_DEFAULTS config), useTheme and usePageTheme hooks, dark/tan mode toggle with localStorage persistence |
91+
| `theme.tsx` | Theme context with page-specific defaults (PAGE_THEME_DEFAULTS config), useTheme and usePageTheme hooks, dark/tan mode toggle with localStorage persistence. getThemeClasses returns bgPrimary, bgSecondary, bgCard, bgHover (no bare `bg` property) |
9292
| `source.ts` | Source utilities: getSourceLabel (OC/CC/FD/CX/CR/PI labels, handles cursor alias), getSourceColorClass (themed badge colors for OpenCode/Claude Code/Factory Droid/Codex CLI/Cursor/Pi with purple for codex-cli, violet for cursor-sync/cursor, orange #f97316 for pi) |
9393

9494
## docs/

src/pages/Dashboard.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1392,7 +1392,7 @@ function SessionsView({
13921392

13931393
{/* Session detail - full width on mobile, uses absolute positioning on mobile for proper scrolling */}
13941394
{selectedSession && (
1395-
<div className={cn("absolute inset-0 lg:relative lg:inset-auto h-full min-h-0 w-full lg:w-1/2 flex flex-col overflow-hidden z-10 lg:z-auto", t.bg)}>
1395+
<div className={cn("absolute inset-0 lg:relative lg:inset-auto h-full min-h-0 w-full lg:w-1/2 flex flex-col overflow-hidden z-10 lg:z-auto", t.bgPrimary)}>
13961396
{/* Detail header */}
13971397
<div className={cn("px-4 py-3 border-b flex items-center justify-between", t.border)}>
13981398
<div className="flex items-center gap-3 min-w-0">

task.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,13 @@ OpenSync supports two AI coding tools: **OpenCode** and **Claude Code**.
88

99
- [ ] (add next task here)
1010

11+
## Recently Completed (Netlify Build Fix - Theme Property)
12+
13+
- [x] Fixed Netlify build error: `t.bg` property does not exist on theme classes
14+
- Root cause: Dashboard.tsx line 1395 used `t.bg` but theme classes only have `bgPrimary`
15+
- Fix: Changed `t.bg` to `t.bgPrimary` in session detail container className
16+
- TypeScript and Netlify build now pass
17+
1118
## Recently Completed (Issue #13 - Mobile Scrolling Fix)
1219

1320
- [x] Fixed mobile scrolling in session detail view (GitHub #13)

0 commit comments

Comments
 (0)