Merge pull request #659 from Orfium/fix/NDS-772_select_option_click_h…

…andler_not_triggered
Orfium · Aug 11, 2023 · 44c45b8 · 44c45b8
2 parents 2f82922 + 4554081
commit 44c45b8
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 7 deletions.
diff --git a/.storybook/preview.tsx b/.storybook/preview.tsx
@@ -131,4 +131,5 @@ export const parameters = {
     viewports: viewPorts,
   },
   options: { showPanel: true },
+  chromatic: { delay: 2000 },
 };
diff --git a/src/components/Chart/Chart.stories.mdx b/src/components/Chart/Chart.stories.mdx
@@ -8,12 +8,7 @@ import DonutChartShowCase from '../storyUtils/DonutChartShowCase';
 import { color } from './LineChart/story.utils';
 import { initData } from './LineChart/mockedData';
 
-<Meta
-  title="Design System/Chart"
-  parameters={{
-    chromatic: { delay: 2000 },
-  }}
-/>
+<Meta title="Design System/Chart" />
 
 # LineChart
 

diff --git a/src/components/Select/Select.tsx b/src/components/Select/Select.tsx
@@ -310,8 +310,11 @@ const Select = React.forwardRef<HTMLInputElement, Props & InputProps & TestProps
       const isEnter = event.key === 'Enter';
 
       if (hasNoOptionsAndIsCreatable && isEnter) {
-        handleMultiSelectOptionClick(filteredOptions[0]);
+        const selectedOption = filteredOptions[0];
+
+        handleMultiSelectOptionClick(selectedOption);
         setSearchValue('');
+        handleSelectedOption(selectedOption);
       }
     };
 

diff --git a/src/components/Table/Table.stories.mdx b/src/components/Table/Table.stories.mdx
@@ -59,6 +59,19 @@ import { Table } from '@orfium/ictinus';
 
 <Props of={Table} />
 
+### Disclaimer 
+
+In case of importing data produced using useMemo or external const, follow this pattern:
+
+```js
+import { Row } from '@orfium/ictinus/dist/components/Table/Table';
+...
+const data: Row<DataType> = useMemo(() => {}, []);
+//or other types of data
+```
+
+Where Row is the type from the Table, so you must import it from there.
+
 # Regular Table
 
 Regular Table with simple rows