feat: add "/" keyboard shortcut to focus search#1807
Conversation
|
@PalashKulkarni is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
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 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
|
✅ Synchronized metadata from Issue #1800:
|
|
Hi @maintainers, I've completed the implementation for Issue #1800 and submitted a PR for review. Summary of Changes
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! |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
sanjay-kv
left a comment
There was a problem hiding this comment.
check the deployment its aint working.
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
Changes Made
Added a global
keydownlistener 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:
inputtextareaselectcontenteditableelementsPrevents 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
Checklist
npm run buildand attached screenshot(s) in this PR.