fix(v2): remove empty doc sidebar container (#1870)

facebook · Oct 23, 2019 · 11f9977 · 11f9977
1 parent 68d5adf
commit 11f9977
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/CHANGELOG-2.x.md b/CHANGELOG-2.x.md
@@ -3,6 +3,7 @@
 ## Unreleased
 
 - Add sticky footer.
+- Remove empty doc sidebar container
 
 ## 2.0.0-alpha.30
 

diff --git a/packages/docusaurus-theme-classic/src/theme/DocPage/index.js b/packages/docusaurus-theme-classic/src/theme/DocPage/index.js
@@ -23,13 +23,15 @@ function DocPage(props) {
   return (
     <Layout noFooter>
       <div className={styles.docPage}>
-        <div className={styles.docSidebarContainer}>
-          <DocSidebar
-            docsSidebars={docsSidebars}
-            location={location}
-            sidebar={sidebar}
-          />
-        </div>
+        {sidebar && (
+          <div className={styles.docSidebarContainer}>
+            <DocSidebar
+              docsSidebars={docsSidebars}
+              location={location}
+              sidebar={sidebar}
+            />
+          </div>
+        )}
         <main className={styles.docMainContainer}>
           <MDXProvider components={MDXComponents}>
             {renderRoutes(route.routes)}