Bug/#363 on change double trigger (#364)

* Removed onClick from expand row cell * package-lock updated
ZEISS · Feb 7, 2022 · 78acd98 · 78acd98
1 parent 8cfe932
commit 78acd98
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 10 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Precise UI Changelog
 
+## 2.1.7
+
+- Fix `onChange` double trigger on clicking `AccordionTable` expand icon
+
 ## 2.1.6
 
 - Fixed controlled mode on Pagination component

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "precise-ui",
-  "version": "2.1.6",
+  "version": "2.1.7",
   "description": "Precise UI React component library powered by Styled Components.",
   "keywords": [
     "react",
@@ -125,4 +125,4 @@
   "engines": {
     "node": ">=10.0.0"
   }
-}
+}
diff --git a/src/components/AccordionTable/AccordionTableBasic.part.tsx b/src/components/AccordionTable/AccordionTableBasic.part.tsx
@@ -297,13 +297,7 @@ export class AccordionTableBasic<T> extends React.Component<AccordionTableProps<
     const { row, data } = e;
 
     if (e.column === 0) {
-      return (
-        <StyledIcon
-          isRotated={hasIndex(this.state.selectedIndex, row)}
-          name="KeyboardArrowRight"
-          onClick={() => data && this.handleClick(row, data)}
-        />
-      );
+      return <StyledIcon isRotated={hasIndex(this.state.selectedIndex, row)} name="KeyboardArrowRight" />;
     }
 
     const { cellRenderer } = this.props;