Skip to content

feat: add "/" keyboard shortcut to focus search#1807

Closed
PalashKulkarni wants to merge 1 commit into
recodehive:mainfrom
PalashKulkarni:feat/add-shortcut
Closed

feat: add "/" keyboard shortcut to focus search#1807
PalashKulkarni wants to merge 1 commit into
recodehive:mainfrom
PalashKulkarni:feat/add-shortcut

Conversation

@PalashKulkarni

Copy link
Copy Markdown

Description

Adds a global / keyboard shortcut to quickly focus the website search bar across documentation and blog pages. This improves navigation speed and accessibility by allowing users to start searching without manually clicking the search input.

Fixes #1800

Type of Change

  • New feature (e.g., new page, component, or functionality)
  • Bug fix (non-breaking change that fixes an issue)
  • UI/UX improvement (design, layout, or styling updates)
  • Performance optimization (e.g., code splitting, caching)
  • Documentation update (README, contribution guidelines, etc.)
  • Other (please specify):

Changes Made

  • Added a global keydown listener to handle the / keyboard shortcut.

  • Focuses the search input when / is pressed anywhere on the site.

  • Supports both documentation and blog search inputs.

  • Prevents the shortcut from triggering while the user is typing inside:

    • input
    • textarea
    • select
    • contenteditable elements
  • Prevents the / character from being inserted when the shortcut activates.

  • Added proper cleanup for event listeners to avoid duplicate handlers and memory leaks.

  • Preserved all existing search functionality and keyboard interactions.

Dependencies

  • No new dependencies added.
  • No version updates required.
  • No additional configuration changes required.

Checklist

  • My code follows the style guidelines of this project.
  • I have tested my changes across major browsers and devices.
  • My changes do not generate new console warnings or errors.
  • I ran npm run build and attached screenshot(s) in this PR.
  • This is already assigned Issue to me, not an unassigned issue.

@vercel

vercel Bot commented May 31, 2026

Copy link
Copy Markdown
Contributor

@PalashKulkarni is attempting to deploy a commit to the recode Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs.

In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊

Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv.

We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰

🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨

📚Your perks for contribution to this community 👇🏻

  1. Get free Consultation use code recode50 to get free: Mentorship for free.

  2. Get the Ebook for free use code recode at checkout: Data Science cheatsheet for Beginners.

  3. Check out this weekly Newsletter: Sanjay's Newsletter.

If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@github-actions github-actions Bot added in-review The current changes are in review and would need approval and testing before merging level 1 10 points recode this is label for leaderboard labels May 31, 2026
@github-actions github-actions Bot added this to the recode:launch 3.0 milestone May 31, 2026
@github-actions github-actions Bot added Enhacement gssoc26 label gssoc:approved gssoc label high priority gssoc26 label level2 gssoc26 label medium gssoc26 label labels May 31, 2026
@github-actions

Copy link
Copy Markdown

✅ Synchronized metadata from Issue #1800:

  • Labels: level 1, recode, Enhacement, high priority, level2, medium, gssoc:approved
  • Milestone: recode:launch 3.0

@PalashKulkarni

Copy link
Copy Markdown
Author

Hi @maintainers,

I've completed the implementation for Issue #1800 and submitted a PR for review.

Summary of Changes

  • Added a global / keyboard shortcut to focus the search bar.
  • Supports both documentation and blog search inputs.
  • Prevents triggering while typing in inputs, textareas, selects, or contenteditable elements.
  • Preserves existing search functionality and keyboard interactions.
  • Includes proper event listener cleanup to avoid memory leaks.

I've tested the feature locally and verified the expected behavior across the relevant pages.

Could you please review the PR and let me know if any changes are needed? If everything looks good, I'd appreciate it if it could be merged.

Thank you for your time and feedback!

@vercel

vercel Bot commented May 31, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
recode-website Ready Ready Preview, Comment May 31, 2026 4:18am

@sanjay-kv sanjay-kv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check the deployment its aint working.

@iitzIrFan iitzIrFan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already have Ctrl + K for this use case, so adding another entry point would duplicate functionality without much benefit.

@iitzIrFan iitzIrFan added the don't-merge faced conflict issue or other dev related issue, dont merge PR label Jun 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

don't-merge faced conflict issue or other dev related issue, dont merge PR Enhacement gssoc26 label gssoc:approved gssoc label high priority gssoc26 label in-review The current changes are in review and would need approval and testing before merging level 1 10 points level2 gssoc26 label medium gssoc26 label recode this is label for leaderboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Add "/" Keyboard Shortcut to Focus Documentation Search

3 participants