From a80aa493a57178237b2a607584c130f9a75ed5aa Mon Sep 17 00:00:00 2001
From: everdimension <everdimension@gmail.com>
Date: Thu, 12 Dec 2019 13:01:59 +0300
Subject: [PATCH] Sidebar pattern update

`overflow: auto` is usually a better choice than `overflow: scroll` when you want to make component scrollable. The only difference is that with `auto` you do not see the scrollbar when there's nothing to scroll.
---
 client/patterns/sidebar/Details.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/client/patterns/sidebar/Details.tsx b/client/patterns/sidebar/Details.tsx
index 1a9a91c..d97b713 100644
--- a/client/patterns/sidebar/Details.tsx
+++ b/client/patterns/sidebar/Details.tsx
@@ -28,7 +28,7 @@ const Details: React.FC<{}> = () => {
                             <div
                                 style={{
                                     flex: 1,
-                                    overflow: 'scroll',
+                                    overflow: 'auto',
                                     padding: '16px',
                                 }}
                             >
@@ -53,7 +53,7 @@ const Details: React.FC<{}> = () => {
         flex: 1;
 
         /* Make it scrollable */
-        overflow: scroll;
+        overflow: auto;
     ">
         ...
     </main>