qwik icon indicating copy to clipboard operation
qwik copied to clipboard

[🐞] Docs: conflicting ctrl+k shortcut

Open AustinGil opened this issue 2 years ago • 2 comments

Which component is affected?

Qwik Playground

Describe the bug

In the Qwik docs, there is a keyboard binding for ctrl+k that brings up the search box, but it also conflicts with the default browser behavior of focusing on the browser search/url bar.

It would be nice to remove the default behavior and autofocus the docs search widget.

Reproduction

https://qwik.builder.io/docs/

Steps to reproduce

Open docs press ctrl+k

System Info

Chrome, Edge, Firefox

Additional Information

No response

AustinGil avatar Jul 03 '23 15:07 AustinGil

I'm on a Mac and I can perfectly focus on the address bar using CMD + L in Chrome. In Windows or Linux there are also other alternatives than CTRL + K, like CTRL + L (link).

JoepKockelkorn avatar Feb 15 '24 14:02 JoepKockelkorn

The issue is not in focusing the address bar. That actually works as expected. However, I think the intended behavior is that ctrl+k should open the in-app search box and focus it.

The actual behavior is that it opens in in-app search, but then the native key bindings move the focus to the search bar, which opens it instead and obscures the in-app search. This default behavior should be prevented, imo.

c04a38e7-7ff8-42b9-8500-880c049d3c5b

AustinGil avatar Feb 15 '24 15:02 AustinGil