Skip to content

Commit cb9b30d

Browse files
committed
fix(web): match stop/running chip sizes, fix AppShell flex layout
- Stop button uses Chip component (same as status) for consistent size - AppShell main is flex-col so children can use h-full - Remove overflow-hidden from root (was blocking dashboard scroll) - Add overflow-auto on main for pages that need scrolling
1 parent 3545518 commit cb9b30d

2 files changed

Lines changed: 6 additions & 7 deletions

File tree

apps/web/src/features/session/ContainerSession.tsx

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -475,14 +475,13 @@ export default function ContainerSession({
475475
<span data-testid="session-status"><Chip variant={statusVariant}>{status}</Chip></span>
476476

477477
{isRunning && (
478-
<button
478+
<span
479479
onClick={handleStop}
480-
disabled={stopping}
481480
data-testid="stop-button"
482-
className="font-mono text-[11px] font-medium px-2 py-0.5 rounded-default border border-danger/35 text-danger bg-danger/10 hover:bg-danger/20 transition-colors cursor-pointer disabled:opacity-40"
481+
className={`cursor-pointer hover:brightness-125 transition-all ${stopping ? 'opacity-40 pointer-events-none' : ''}`}
483482
>
484-
{stopping ? 'stopping...' : 'stop'}
485-
</button>
483+
<Chip variant="danger">{stopping ? 'stopping...' : 'stop'}</Chip>
484+
</span>
486485
)}
487486
</div>
488487
</div>

apps/web/src/shared/components/AppShell.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ interface AppShellProps {
77

88
export function AppShell({ children }: AppShellProps) {
99
return (
10-
<div className="relative h-screen flex flex-col bg-bg text-ink overflow-hidden">
10+
<div className="relative h-screen flex flex-col bg-bg text-ink">
1111
<GrainOverlay />
1212
<Topbar />
13-
<main className="relative flex-1 min-h-0 max-w-[1120px] w-full mx-auto px-5 py-6">
13+
<main className="relative flex-1 min-h-0 flex flex-col max-w-[1120px] w-full mx-auto px-5 py-6 overflow-auto">
1414
{children}
1515
</main>
1616
</div>

0 commit comments

Comments
 (0)