refactor(website/responsive): make the website responsive - code review

adeo · Sep 1, 2020 · 74871f3 · 74871f3
1 parent 14c018b
commit 74871f3
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 6 deletions.
diff --git a/packages/gatsby-theme-styleguide/src/gatsby-components/TableOfContents/tableofcontents.scss b/packages/gatsby-theme-styleguide/src/gatsby-components/TableOfContents/tableofcontents.scss
@@ -13,12 +13,15 @@
   }
 
   &__wrapper {
-    border-left: 1px solid #887f87;
-    max-height: 90vh;
-    overflow-y: auto;
     padding: 1rem;
-    position: sticky;
-    top: 0;
+
+    @media screen and (min-width: 1280px) {
+      border-left: 1px solid #887f87;
+      max-height: 90vh;
+      overflow-y: auto;
+      position: sticky;
+      top: 0;
+    }
   }
 
   &__title {

diff --git a/packages/gatsby-theme-styleguide/src/gatsby-components/base.scss b/packages/gatsby-theme-styleguide/src/gatsby-components/base.scss
@@ -102,3 +102,7 @@ th:first-child,
 td:first-child {
   padding-left: 0;
 }
+
+.nav-open {
+  overflow: hidden;
+}
diff --git a/packages/gatsby-theme-styleguide/src/templates/pattern-page.js b/packages/gatsby-theme-styleguide/src/templates/pattern-page.js
@@ -149,7 +149,7 @@ export default ({ data, location }) => {
   const hasTabs = samePageTabs.length > 1
   const hasMainCategory = mainCategory.length > 3
 
-  function handleMenu(e) {
+  const handleMenu = () => {
     if (document.body.classList.contains('nav-open')) {
       document.body.classList.remove('nav-open')
       return false