atlantis icon indicating copy to clipboard operation
atlantis copied to clipboard

refactor(components): Adjust Combobox onSearch Prop Logic

Open ZakaryH opened this issue 1 year ago • 1 comments

Motivations

I wasn't the biggest fan of passing around a noop and eventually calling it with some arguments

tried to think of a way to do it that I liked more. this is hopefully that.

Changes

really just swapped from it being a noop in the case where we don't need an extra on search callback to being undefined and now we won't attempt to call this method if it was undefined

then some minor name and internal prop doc updates

oh and changed the debounced function from being a useCallback that gets invoked on each re-render to a useMemo that won't as per a suggestion from Jacob!

Added

Changed

Deprecated

Removed

Fixed

Security

Testing


In Atlantis we use Github's built in pull request reviews.

Random photo of Atlantis

ZakaryH avatar Feb 02 '24 00:02 ZakaryH