Skip to content

Search input debounce#712

Merged
kentcdodds merged 2 commits intomainfrom
cursor/search-input-debounce-2134
Feb 26, 2026
Merged

Search input debounce#712
kentcdodds merged 2 commits intomainfrom
cursor/search-input-debounce-2134

Conversation

@kentcdodds
Copy link
Owner

@kentcdodds kentcdodds commented Feb 26, 2026

Debounce search on the /search page to 200ms to reduce cost and align with the nav autocomplete debounce.


Open in Web Open in Cursor 


Note

Low Risk
Low risk UI tweak that only changes the timing of search suggestion requests and should not affect data correctness.

Overview
Adjusts the navbar autocomplete behavior by increasing the useDebounce delay for /resources/search suggestion loading from 200ms to 250ms, slightly reducing request frequency while typing.

Written by Cursor Bugbot for commit 483bd2f. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

Release Notes

  • Performance
    • Refined search suggestions loading timing for a more optimized search experience.

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor
Copy link

cursor bot commented Feb 26, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 733a15a and 483bd2f.

📒 Files selected for processing (1)
  • app/components/navbar.tsx

📝 Walkthrough

Walkthrough

The pull request adjusts the debounce delay for search suggestion loading in the NavSearch component from 200ms to 250ms. This timing adjustment controls when the fetcher initiates data retrieval following user input in the search field.

Changes

Cohort / File(s) Summary
Search Debounce Timing
app/components/navbar.tsx
Increased debounce delay from 200ms to 250ms for search suggestion fetcher.load invocation in NavSearch component.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • Downshift getMenuProps call #616: Modifies the NavSearch implementation in the same file with restructuring of Downshift props and suggestion display computation logic.

Poem

🐰 A rabbit hops through search delays,
Two-fifty feels just right today,
The suggestions come a touch more slow,
But smoother interactions steal the show! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Search input debounce' directly summarizes the main change—increasing the debounce delay for search input suggestions from 200ms to 250ms.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cursor/search-input-debounce-2134

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kentcdodds kentcdodds marked this pull request as ready for review February 26, 2026 22:38
@kentcdodds kentcdodds merged commit eed6527 into main Feb 26, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants