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

Using "/" to focus search box does not work in Firefox #10646

Closed
apollo13 opened this issue May 23, 2021 · 7 comments · Fixed by #10714
Closed

Using "/" to focus search box does not work in Firefox #10646

apollo13 opened this issue May 23, 2021 · 7 comments · Fixed by #10714

Comments

@apollo13
Copy link
Contributor

apollo13 commented May 23, 2021

Typing "/" in Firefox (german keyboard layout, but that shouldn't make a difference) does not focus the search box but rather brings up Firefox's builtin quick search. See https://support.mozilla.org/en-US/kb/keyboard-shortcuts-perform-firefox-tasks-quickly#w_search -- I am not sure if there is a good way to override it.

Nomad version is 1.1.0

@tgross tgross added this to Needs Triage in Nomad - Community Issues Triage via automation May 24, 2021
@tgross
Copy link
Member

tgross commented Jun 7, 2021

Thanks @apollo13. Can you provide a bit more information about your environment to help us debug this? I wasn't able to confirm this on macOS with Firefox 89.0 with an English/US keyboard layout.

@tgross tgross self-assigned this Jun 7, 2021
@tgross tgross moved this from Needs Triage to In Progress in Nomad - Community Issues Triage Jun 7, 2021
@apollo13
Copy link
Contributor Author

apollo13 commented Jun 7, 2021

Mhm, Fedora 34 with Firefox 88.0.1 and german keyboard layout. (Works on Github if I hit "shift + 7" which is "/" on german layout)

@tgross
Copy link
Member

tgross commented Jun 7, 2021

Ok, thanks. We'll get this looked at.

@tgross tgross removed this from In Progress in Nomad - Community Issues Triage Jun 7, 2021
@tgross tgross removed their assignment Jun 7, 2021
@apollo13
Copy link
Contributor Author

apollo13 commented Jun 7, 2021

I can confirm that it works after switching to an english keyboard layout.

@apollo13
Copy link
Contributor Author

apollo13 commented Jun 7, 2021

I think this could be fixed by changing https://github.com/hashicorp/nomad/blob/main/ui/app/components/global-search/control.js#L27 to use code == "/". Let me check.

@apollo13
Copy link
Contributor Author

apollo13 commented Jun 7, 2021

I have attached a PR that seems to fix it for me, reviews welcome.

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants