Add onChange event handler to TableSelectRow (#7293)

* Add onChange event handler to TableSelectRow It allows to pass a function to the TableSelectRow when the row was selected. Only tested it with the radio button version. * feat(TableSelectRow): add onChange prop-type * chore(tests): update snapshots * fix(data-table): small tweak Co-authored-by: TJ Egan <tw15egan@gmail.com> Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
carbon-design-system · Nov 30, 2020 · d804515 · d804515
1 parent 3c3049f
commit d804515
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap b/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap
@@ -1250,6 +1250,9 @@ Map {
           "isRequired": true,
           "type": "string",
         },
+        "onChange": Object {
+          "type": "func",
+        },
         "onSelect": Object {
           "isRequired": true,
           "type": "func",
@@ -1870,6 +1873,9 @@ Map {
         "isRequired": true,
         "type": "string",
       },
+      "onChange": Object {
+        "type": "func",
+      },
       "onSelect": Object {
         "isRequired": true,
         "type": "func",

diff --git a/packages/react/src/components/DataTable/TableSelectRow.js b/packages/react/src/components/DataTable/TableSelectRow.js
@@ -19,6 +19,7 @@ const TableSelectRow = ({
   id,
   name,
   onSelect,
+  onChange,
   disabled,
   radio,
   className,
@@ -27,6 +28,7 @@ const TableSelectRow = ({
     id,
     name,
     onClick: onSelect,
+    onChange,
     checked,
     disabled,
   };
@@ -79,6 +81,11 @@ TableSelectRow.propTypes = {
    */
   name: PropTypes.string.isRequired,
 
+  /**
+   * Provide an optional hook that is called each time the input is updated
+   */
+  onChange: PropTypes.func,
+
   /**
    * Provide a handler to listen to when a user initiates a selection request
    */