Sébastien Beaury

Results 4 comments of Sébastien Beaury

hey @Djunnni, the purpose of this PR is actually to remove the extra 30px from the navbar height (100vh) coming from `--docusaurus-announcement-bar-height` to avoid things getting squeezed at the bottom....

As mentioned above, my PR was to fix the overlap when announcementBar is visible. What we know for sure is that `.sidebarViewport` is too high when announcement bar is visible...

Last [commit](https://github.com/facebook/docusaurus/pull/8493/commits/3980a31b42f3b660859026e853beb30dd68c5196) should fix the problem hopefully 🤞. I dynamically subtracted the portion of the announcement bar visible on viewport from the sidebar container's max-height.

you're right I forgot to add a conditional using `useIsBrowser` for client side only but I finally opted out for another solution in my last commit which is to declare...