From d8045151dd2706c3cd2cef45acdbb52a7513d0cd Mon Sep 17 00:00:00 2001 From: Zsolt Lattmann Date: Mon, 30 Nov 2020 14:50:09 -0800 Subject: [PATCH] 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 Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com> --- .../react/__tests__/__snapshots__/PublicAPI-test.js.snap | 6 ++++++ packages/react/src/components/DataTable/TableSelectRow.js | 7 +++++++ 2 files changed, 13 insertions(+) diff --git a/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap b/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap index 03fa4526106e..d80686786d6e 100644 --- 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 index 572dd93c1715..ef9e5d2e492e 100644 --- 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 */