global-header NFS wrapper obscures the first sidebar item #28

Activity

1. fullsend-ai-triage commented on Sep 3, 2026

   

   fullsend-ai-triageBot

   on Sep 3, 2026 – with fullsend-ai-triage

   Last edited by fullsend-ai-triage

   More actions

   🤖 Finished Triage · ✅ Success · Started 1:32 PM UTC · Completed 1:38 PM UTC

   Commit: 45fe3a5 · View workflow run →

   Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $1.31
2. fullsend-ai-triage

   added

   bugSomething isn't workingSomething isn't working

   ready-to-code

   on Sep 3, 2026
3. fullsend-ai-triage commented on Sep 3, 2026

   

   fullsend-ai-triageBot

   on Sep 3, 2026 – with fullsend-ai-triage

   More actions

   Root cause hypothesis

   The NFS GlobalHeaderWrapper (src/extensions/globalHeaderModule.tsx) uses AppRootWrapperBlueprint to render a sticky AppBar above {children}. The sidebar sits inside those children and starts at the viewport top without accounting for the header height, so the first item gets hidden behind it. The legacy frontend system avoids this because the header is mounted differently — as a component extension within the layout rather than as a root wrapper.

   Notably, the workspace README documents a position config option (above-sidebar | above-main-content), but this does not appear to be wired into the NFS wrapper code — implementing it may be part of the fix.

   Recommended fix

   Ensure the sidebar content is offset below the header. Options include a flex-based column layout (header then sidebar+content), a CSS custom property for the header height, or a top offset on the children container. The position config option should also be implemented in the NFS code path if it isn't already.

   Proposed test case

   // Unit: verify the wrapper applies offset styling
   render(
     <GlobalHeaderWrapper extensionComponents={[]} extensionMenuItems={[]}>
       <div data-testid="sidebar-first-item">Home</div>
     </GlobalHeaderWrapper>
   );
   const header = document.getElementById('global-header');
   const firstItem = screen.getByTestId('sidebar-first-item');
   expect(firstItem.getBoundingClientRect().top)
     .toBeGreaterThanOrEqual(header.getBoundingClientRect().bottom);

   ---

   Next steps:

   • /fs-code — agent creates a PR to implement this issue
   • /fs-code <your instruction> — agent implements with your specific guidance
4. kadel commented on Sep 3, 2026

   

   kadel

   on Sep 3, 2026

   Author

   More actions

   /fs-code
5. fullsend-ai-coder commented on Sep 3, 2026

   

   fullsend-ai-coderBot

   on Sep 3, 2026 – with fullsend-ai-coder

   Last edited by fullsend-ai-coder

   More actions

   🤖 Finished Code · ✅ Success · Started 1:40 PM UTC · Completed 1:59 PM UTC

   Commit: 45fe3a5 · View workflow run →

   Runtime: claude · Model: claude-opus-4-6 · Cost: $3.44