fix(striker-ui): separate types from Select component

striker-ui/components/Select.tsx

@@ -1,45 +1,25 @@
-import { FC } from 'react';
+import { Close as CloseIcon } from '@mui/icons-material';
 import {
   IconButton as MUIIconButton,
-  IconButtonProps as MUIIconButtonProps,
   iconButtonClasses as muiIconButtonClasses,
   inputClasses,
   Select as MUISelect,
   selectClasses as muiSelectClasses,
-  SelectProps as MUISelectProps,
   InputAdornment as MUIInputAdornment,
   inputAdornmentClasses as muiInputAdornmentClasses,
 } from '@mui/material';
-import { Close as CloseIcon } from '@mui/icons-material';
+import { FC, useMemo } from 'react';
 
 import { GREY } from '../lib/consts/DEFAULT_THEME';
 
-type SelectOptionalProps = {
+const Select: FC<SelectProps> = ({
-  onClearIndicatorClick?: MUIIconButtonProps['onClick'] | null;
+  onClearIndicatorClick,
-};
-
-type SelectProps = MUISelectProps & SelectOptionalProps;
-
-const SELECT_DEFAULT_PROPS: Required<SelectOptionalProps> = {
-  onClearIndicatorClick: null,
-};
-
-const Select: FC<SelectProps> = (selectProps) => {
-  const {
-    onClearIndicatorClick = SELECT_DEFAULT_PROPS.onClearIndicatorClick,
   ...muiSelectProps
-  } = selectProps;
+}) => {
-  const { children, sx, value } = muiSelectProps;
+  const { sx: selectSx, value, ...restMuiSelectProps } = muiSelectProps;
-  const clearIndicator: JSX.Element | undefined =
-    String(value).length > 0 && onClearIndicatorClick ? (
-      <MUIInputAdornment position="end">
-        <MUIIconButton onClick={onClearIndicatorClick}>
-          <CloseIcon fontSize="small" />
-        </MUIIconButton>
-      </MUIInputAdornment>
-    ) : undefined;
 
-  const combinedSx = {
+  const combinedSx = useMemo(
+    () => ({
       [`& .${muiSelectClasses.icon}`]: {
         color: GREY,
       },
@@ -59,24 +39,32 @@ const Select: FC<SelectProps> = (selectProps) => {
           visibility: 'visible',
         },
 
-    ...sx,
+      ...selectSx,
-  };
+    }),
+    [selectSx],
+  );
+
+  const clearIndicatorElement = useMemo(
+    () =>
+      String(value).length > 0 &&
+      onClearIndicatorClick && (
+        <MUIInputAdornment position="end">
+          <MUIIconButton onClick={onClearIndicatorClick}>
+            <CloseIcon fontSize="small" />
+          </MUIIconButton>
+        </MUIInputAdornment>
+      ),
+    [onClearIndicatorClick, value],
+  );
 
   return (
     <MUISelect
-      {...{
+      endAdornment={clearIndicatorElement}
-        endAdornment: clearIndicator,
+      value={value}
-        ...muiSelectProps,
+      {...restMuiSelectProps}
-        sx: combinedSx,
+      sx={combinedSx}
-      }}
+    />
-    >
-      {children}
-    </MUISelect>
   );
 };
 
-Select.defaultProps = SELECT_DEFAULT_PROPS;
-
-export type { SelectProps };
-
 export default Select;

striker-ui/types/Select.d.ts

@@ -0,0 +1,5 @@
+type SelectOptionalProps = {
+  onClearIndicatorClick?: import('@mui/material').IconButtonProps['onClick'];
+};
+
+type SelectProps = import('@mui/material').SelectProps & SelectOptionalProps;