From f085560b66c97b9dc0b9eaab73043b80eb7c2578 Mon Sep 17 00:00:00 2001
From: Alexey Pyltsyn <lex61rus@gmail.com>
Date: Wed, 28 Oct 2020 19:52:44 +0300
Subject: [PATCH] fix(v2): do not set height for mobile dropdown (#3647)

---
 .../src/theme/NavbarItem/DefaultNavbarItem.tsx            | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx b/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx
index 3f160b3b81..2ba2a7e495 100644
--- a/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx
+++ b/packages/docusaurus-theme-classic/src/theme/NavbarItem/DefaultNavbarItem.tsx
@@ -179,6 +179,10 @@ function NavItemMobile({
     );
   }
 
+  const menuListHeight = menuListRef.current?.scrollHeight
+    ? `${menuListRef.current?.scrollHeight}px`
+    : undefined;
+
   return (
     <li
       className={clsx('menu__list-item', {
@@ -197,9 +201,7 @@ function NavItemMobile({
         className="menu__list"
         ref={menuListRef}
         style={{
-          height: !collapsed
-            ? `${menuListRef.current?.scrollHeight}px`
-            : undefined,
+          height: !collapsed ? menuListHeight : undefined,
         }}>
         {items.map(({className: childItemClassName, ...childItemProps}, i) => (
           <li className="menu__list-item" key={i}>