Skip to content

Commit

Permalink
fix: tabletoolbarsearch type for onchange (#15395)
Browse files Browse the repository at this point in the history
There was obviously some confusion between functions and
the events themselves.

There are a lot of other event handlers that need to be
cleaned up, especially the ones defined like (evt: any) => void,
but this just addresses TableToolbarSearch.

Refs #14471.

Co-authored-by: Andrea N. Cardona <cardona.n.andrea@gmail.com>
  • Loading branch information
wkeese and andreancardona authored Jan 30, 2024
1 parent 173e576 commit 51d91f2
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
import cx from 'classnames';
import PropTypes from 'prop-types';
import React, {
ChangeEvent,
useMemo,
useRef,
useState,
Expand Down Expand Up @@ -83,7 +84,7 @@ export interface TableToolbarSearchProps {
* Provide an optional hook that is called each time the input is updated
*/
onChange?: (
event: '' | Partial<React.ChangeEventHandler<HTMLInputElement>>,
event: '' | ChangeEvent<HTMLInputElement>,
value?: string
) => void;

Expand Down

0 comments on commit 51d91f2

Please sign in to comment.