fix: added fix for split btn icon

EightfoldAI · Aug 19, 2022 · 423fe84 · 423fe84
1 parent 98f1f70
commit 423fe84
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/src/components/Button/Button.stories.tsx b/src/components/Button/Button.stories.tsx
@@ -347,6 +347,11 @@ Split.args = {
     split: true,
     splitButtonChecked: false,
     text: 'Split Button',
+    splitButtonProps: {
+        iconProps: {
+            path: IconName.mdiClockOutline,
+        },
+    },
 };
 
 Split_With_Counter.args = {

diff --git a/src/components/Button/SplitButton/SplitButton.tsx b/src/components/Button/SplitButton/SplitButton.tsx
@@ -134,15 +134,18 @@ export const SplitButton: FC<SplitButtonProps> = React.forwardRef(
             return iconSize;
         };
 
+        const getButtonIconPath = (): IconName =>
+            iconProps?.path
+                ? iconProps.path
+                : checked
+                ? IconName.mdiChevronUp
+                : IconName.mdiChevronDown;
+
         const getButtonIcon = (): JSX.Element => (
             <Icon
                 {...iconProps}
                 classNames={styles.icon}
-                path={
-                    iconProps?.path || checked
-                        ? IconName.mdiChevronUp
-                        : IconName.mdiChevronDown
-                }
+                path={getButtonIconPath()}
                 size={getButtonIconSize()}
             />
         );