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:
parent
9b8947a198
commit
fb3412c9a5
|
|
@ -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}
|
||||
|
|
|
|||
Loading…
Reference in New Issue