Skip to content

Commit 00aa59f

Browse files
committed
fix: match advanced FEN page height model to the rest of the shell
This page was the only PageSidebarLayout consumer still using h-screen/max-h-screen with its own data-page-scroll marker, left over from before the shared layout switched to sizing off its parent. Since #main-content is already shorter than 100vh (navbar offset), h-screen made the page taller than its container and created a second, nested scroll context — which broke the sidebar's sticky positioning here specifically while every other tab page (About, Settings, Export, FEN History) stayed fixed correctly. Switched it to md:h-full/md:max-h-full like the rest so the sidebar and separator behave the same everywhere.
1 parent e18d541 commit 00aa59f

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

src/pages/AdvancedFENInputPage/AdvancedFENInputPage.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,10 +71,7 @@ const AdvancedFENInputPage = memo(function AdvancedFENInputPage(
7171
);
7272

7373
return (
74-
<div
75-
data-page-scroll
76-
className="bg-bg min-h-full md:h-screen md:max-h-screen md:overflow-hidden"
77-
>
74+
<div className="bg-bg min-h-full md:h-full md:max-h-full">
7875
<Seo
7976
{...getRouteSeo('/advanced-fen')}
8077
schema={[SOFTWARE_APP_SCHEMA, ADVANCED_FEN_BREADCRUMB_SCHEMA]}

0 commit comments

Comments
 (0)