You can no longer press ctrl+f twice to use native Find

I’m not sure when this changed but it used to be that ctrl+f once would bring up Discourse’s Find feature and pressing it a second time would bring up the browser’s native Find feature. I thought this was quite useful, but now the behaviour has changed and the second press closes Discourse’s Find and does nothing else.

1개의 좋아요

The searc function, can be active with /.

1개의 좋아요

No, this is a legitimate bug. On long topics, pressing CTRL + F will open the Discourse Search, press it again and it should open the native browser search.

PR of it working previously

3개의 좋아요

Yeah, that’s a regression. I think I know what the problem is… will fix it tonight.

2개의 좋아요

I bisected to this commit. Migrate search drop down to `menu-panel` component. · discourse/discourse@d4b987f · GitHub

We should probably add an acceptance tests too :blush:

1개의 좋아요

I think this should fix it:
https://github.com/discourse/discourse/pull/3844

I have no idea how to do this. Is it possible to check if the browser handled the event?
You can add it if you want. :wink: Would be great to see how this can be done.

8개의 좋아요

Thanks, all! Good job.

For anybody being bugged by the Ctrl+f key combination not working on some other websites which have taken it over…

I have found myself getting used to pressing Alt+d, Ctrl+f.

Alt+d puts you in the address bar, from there the search shortcut works.
(tested in Chrome).

6개의 좋아요

Merged! Thanks for the fix @gerhard.

6개의 좋아요

Confirmed works, thanks again!

3개의 좋아요