Sébastien Beaury
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...