fix(core): lazy-load usePopover in SideNavHeading (#264)

[jibin7jose]

Summary

This PR resolves issue #264 by lazy-loading the heavy usePopover and Layer engine resources inside the SideNavHeading component.

Changes Made

• Extracted shared CSS into SideNavHeading.stylex.ts to prevent circular dependencies.
• Extracted static, non-interactive heading UI into SideNavHeadingStatic.tsx.
• Extracted the heavy, popover-dependent menu interactions into SideNavHeadingWithMenu.tsx.
• Updated SideNavHeading.tsx to conditionally load SideNavHeadingWithMenu using React lazy() and <Suspense>, falling back to the static rendering logic if no menu prop is provided.

Impact

Previously, SideNavHeading eagerly imported the Popover engine, bloating the initial bundle size for all users—even when no menu was provided. With this change, applications that use a simple, static SideNav without menus will completely skip downloading the popover logic, resulting in a much smaller bundle size and faster load times.

Fixes #264

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
astryx	Ready	Preview, Comment	Jul 25, 2026 4:56am