Fix mobile menu for KeystarUI docs #1472
Conversation
|
| {nextRootScript} | ||
| <style | ||
| dangerouslySetInnerHTML={{ | ||
| __html: /* css */ `html, body { height: 100%; }`, |
There was a problem hiding this comment.
This is so the main content takes up available space so we can push the footer to the bottom of the page without having to rely on viewport units
| Before | After |
|---|---|
 |
 |
| const pathname = usePathname(); | ||
|
|
||
| useEffect(() => { | ||
| addEventListener('popstate', closeSidebar); |
There was a problem hiding this comment.
This wasn't working to close the menu
Current experience:
https://github.com/user-attachments/assets/befa47e4-bfa5-42ef-a220-c5466045c36b
New experience:
https://github.com/user-attachments/assets/7b6bc1f6-da71-4499-892f-2b6615787b04
| eslint: { ignoreDuringBuilds: true }, | ||
| typescript: { ignoreBuildErrors: true }, | ||
| serverExternalPackages: ['esbuild'], | ||
| experimental: { |
There was a problem hiding this comment.
Unrelated, but Next.js was complaining about this experimental flag being deprecated and to use serverExternalPackages instead.
This fixes the mobile menu for the docs, which was not allowing for scrolling.