From 22b5df6341138658e5c12a44bb48e6b1307614a0 Mon Sep 17 00:00:00 2001
From: Joseph Izang <josephizang@gmail.com>
Date: Mon, 3 Jan 2022 14:20:22 +0100
Subject: [PATCH] add fix to comments from @LianaHus

---
 .../src/lib/components/RequiredSection.tsx          | 13 +++++++++----
 .../src/lib/components/Settings.tsx                 |  4 ++--
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/libs/remix-ui/vertical-icons-panel/src/lib/components/RequiredSection.tsx b/libs/remix-ui/vertical-icons-panel/src/lib/components/RequiredSection.tsx
index c78610052b..3ba070d18c 100644
--- a/libs/remix-ui/vertical-icons-panel/src/lib/components/RequiredSection.tsx
+++ b/libs/remix-ui/vertical-icons-panel/src/lib/components/RequiredSection.tsx
@@ -56,10 +56,15 @@ function RequiredSection ({ verticalIconsPlugin, itemContextAction, addActive, r
         removeActive={removeActive}
         itemContextAction={itemContextAction}
       />
-      <Chevron
-        divElementRef={scrollableRef}
-        cssRule={'fa fa-chevron-up remixui_icon-chevron mt-0 mb-0 ml-1 pl-3'}
-      />
+      {
+        scrollableRef.current && scrollableRef.current.scrollHeight > scrollableRef.current.clientHeight
+          ? (
+            <Chevron
+              divElementRef={scrollableRef}
+              cssRule={'fa fa-chevron-up remixui_icon-chevron mt-0 mb-0 ml-1 pl-3'}
+            />
+          ) : null
+      }
     </Fragment>
   )
 }
diff --git a/libs/remix-ui/vertical-icons-panel/src/lib/components/Settings.tsx b/libs/remix-ui/vertical-icons-panel/src/lib/components/Settings.tsx
index 5aee6995b0..f8f0c04698 100644
--- a/libs/remix-ui/vertical-icons-panel/src/lib/components/Settings.tsx
+++ b/libs/remix-ui/vertical-icons-panel/src/lib/components/Settings.tsx
@@ -34,10 +34,10 @@ function Settings ({ scrollableRef, verticalIconsPlugin, itemContextAction, addA
   }, [])
   return (
     <div id="settingsIcons" className="remixui_settings mb-0 flex-grow-0" data-id="vertialIconsSettingsIcons" ref={settingsRef}>
-      <Chevron
+      { scrollableRef.current && scrollableRef.current.scrollHeight > scrollableRef.current.clientHeight ? (<Chevron
         divElementRef={scrollableRef}
         cssRule={'fa fa-chevron-down remixui_icon-chevron mt-0 mb-0 ml-1 pl-3'}
-      />
+      />) : null }
       {Object.keys(verticalIconsPlugin.targetProfileForChange)
         .filter(p => p === 'settings')
         .map(p => (