fix: position shared worksheet banner below app nav

Fix banner positioning to properly sit below the nav bar:
- Changed from `position: fixed; top: 16` to `paddingTop: var(--app-nav-height)`
- Matches the same approach used in AdditionWorksheetClient
- Added `flexShrink: 0` to banner to prevent it from shrinking
- Changed PanelGroup from `overflow: hidden` to `minHeight: 0` for proper flex layout
- Now banner and panels are correctly positioned within PageWithNav

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Thomas Hallock 2025-11-11 12:49:13 -06:00
parent 9b8947a198
commit fb3412c9a5
1 changed files with 6 additions and 5 deletions

View File

@ -238,12 +238,12 @@ export default function SharedWorksheetPage() {
<div
data-component="shared-worksheet-studio"
className={css({
position: 'fixed',
inset: 0,
top: '16', // Account for nav bar
height: '100vh',
bg: isDark ? 'gray.900' : 'gray.50',
paddingTop: 'var(--app-nav-height)',
overflow: 'hidden',
display: 'flex',
flexDirection: 'column',
bg: isDark ? 'gray.900' : 'gray.50',
})}
>
{/* Read-only banner */}
@ -259,6 +259,7 @@ export default function SharedWorksheetPage() {
justifyContent: 'space-between',
gap: '4',
shadow: 'md',
flexShrink: 0,
})}
>
<div className={css({ display: 'flex', alignItems: 'center', gap: '3' })}>
@ -402,7 +403,7 @@ export default function SharedWorksheetPage() {
</div>
{/* Worksheet studio layout */}
<PanelGroup direction="horizontal" className={css({ flex: '1', overflow: 'hidden' })}>
<PanelGroup direction="horizontal" className={css({ flex: '1', minHeight: '0' })}>
{/* Left sidebar - Config controls (read-only) */}
<Panel
defaultSize={25}