-
Notifications
You must be signed in to change notification settings - Fork 185
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
Autofocus search field #352
Conversation
Merge master
pull from upstream
merge upstream master
This took way longer than expected, this SO question was the biggest help in getting here https://stackoverflow.com/questions/34941829/setting-focus-of-an-input-element-in-vue-js |
el.__vue__.focus() | ||
} | ||
}) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please indent with 2 spaces, I may look into how to make the linter catch these in the future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 🎉
Description
Makes the page auto-focus the search field. Closes #351
Motivation and Context
Saves users from having to click in on the search bar
How Has This Been Tested?
Tested locally with
make build_ui && make run
Types of changes
Checklist: