[front] - refacto: add useDebounce hook
#11470
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Introduce a new custom hook to manage debounced input values and states - Allow configuration of debounce delay and minimum string length for updates - Provide immediate and debounced values along with isDebouncing state and control methods
…tion for search feature - Replaced custom debounce implementation with useDebounce hook for search input in InputBarAttachmentsPicker component - Removed unnecessary useEffect and useState for managing debounce, simplifying the component logic - Updated useDebounce hook to use string as the input value type for consistency and removed unnecessary type checks
…rchLayout search input - Integrate `useDebounce` hook to handle search input debounce logic - Replace manual debouncing implementation with `useDebounce` - Implement reset of pagination when debounced search term changes - Add combined loading state to handle debouncing and search state - Change search input value handling to update debounced value through context - Remove old effect for handling debounced search state transition - Refine effect for search hit count to consider new loading state
- Replaced manual debouncing logic with useDebounce hook to handle search input debounce and state - Simplified useEffect and state management related to search input debouncing by integrating useDebounce hook - Removed unnecessary debouncing state variables and useEffect cleanup code
- Replace manual debouncing implementation with useDebounce hook for search filter functionality - Trigger pagination reset when search filter changes using debounced value - Adjust loading state indicators to account for debouncing state in infinite scroll component - Remove manual debouncing code with useEffect and useState in favor of more concise hook usage
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR introduces a reusable
useDebouncehook to standardize debounce handling across search inputs. The hook encapsulates common debounce logic including minimum length checks, loading states, and cleanup. It replaces multiple custom debounce implementations with a single solutionRisk
Moderate, breaking search across different places
Deploy Plan