Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(docs): Make input autofocus in the /icons page #2581

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

danilo-leal
Copy link

What is the purpose of this pull request?

  • New Icon
  • Bug fix
  • New Feature
  • Documentation update
  • Other:

Description

This PR simply adds the autofocus attribute to the Input component on the /icons page. This allows users to immediately type their search query, which then avoids an extra click. Here's a quick video with the before & after:

autofocus-input.mp4

Love Lucide—we use it extensively at Zed! 😃 I basically have the docs tab open every day.

Before Submitting

@github-actions github-actions bot added the 🌍 site Has to do something with the Lucide website label Nov 4, 2024
@ericfennis
Copy link
Member

@danilo-leal Thanks for the PR. Awesome that you use Lucide at Zed!
I will review this behavior. I'm not sure from a UX perspective yet, I will get back to you!

@danilo-leal
Copy link
Author

Absolutely! I'm constantly refreshing https://lucide.dev/icons/, so having the input automatically focused would be convenient so I don't have to reach for the mouse, but definitely feel free to ditch if you think it doesn't make sense 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌍 site Has to do something with the Lucide website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants