Skip to content
This repository has been archived by the owner on Dec 27, 2022. It is now read-only.

Ctrl+F not working. Find from menu does, though. #1219

Closed
hossameldeen opened this issue Oct 20, 2018 · 13 comments
Closed

Ctrl+F not working. Find from menu does, though. #1219

hossameldeen opened this issue Oct 20, 2018 · 13 comments
Assignees
Labels

Comments

@hossameldeen
Copy link

Operation System: Ubuntu 16.04
Beaker version: Source-latest (7bac87c)

Problem

Ctrl+F not working. "Find in page" from menu works, though.

Relevant

Opened it as a separate issue from #1189 as it's a different problem.

And really, it's no big deal for me and not urgent at all :). And thanks again for fixing #1189, that was awesome! and it eliminated a pain point indeed.

@pfrazee
Copy link
Member

pfrazee commented Oct 21, 2018

Thanks for reporting, I'll check into it.

@pfrazee pfrazee added the bug label Oct 21, 2018
@pfrazee pfrazee self-assigned this Oct 21, 2018
@pfrazee
Copy link
Member

pfrazee commented Oct 21, 2018

Huh, I'm not able to reproduce this on Ubuntu 18.04

@hossameldeen
Copy link
Author

I'm not intending to spend time debugging it for now, it's not that much of an issue for me currently. When I have time, will see how to extract useful Ubuntu logs to debug this problem (event listeners on Ubuntu level, .. etc).

For now, to anyone reading this:

  1. If you're experiencing the problem as well, please tell us to make sure it's not a fault at my side.
  2. If you know how to get useful Ubuntu logs to debug this issue, please tell us.

@pujux
Copy link

pujux commented Oct 24, 2018

I would be really interested in the outcome of this issue! Sadly, no idea what could cause this yet.

@pfrazee
Copy link
Member

pfrazee commented Oct 24, 2018

@hossameldeen are there any other shortcuts that dont work, by chance?

@pujux
Copy link

pujux commented Oct 24, 2018

@pfrazee Does beaker internally log out errors? If so, where could one find them?

@pfrazee
Copy link
Member

pfrazee commented Oct 24, 2018

beaker://debug-log/

@pujux
Copy link

pujux commented Oct 24, 2018

Thanks! Since you didn't mention it, I guess, you won't find any errors related to this issue in the log? That was my first thought, but ubuntu's error logs might be more helpful.

@pfrazee
Copy link
Member

pfrazee commented Oct 24, 2018

Probably not. The debug-log is really noisy right now and I might remove some of the dat events because we now have the swarm debugger.

@hossameldeen
Copy link
Author

@pfrazee You're right, some other shortcuts don't work. For the menu shortcuts:

  • Ctrl+N, T work.
  • Ctrl+F, H, O don't.

I don't know what other shortcuts there are, but Ctrl+number and Ctrl+W work.

@pfrazee
Copy link
Member

pfrazee commented Oct 25, 2018

The ones that work seem to be the ones registered using our localshortcut module. The ones that fail are the keyboard accelerators. I'd bet a nickle it's related to electron/electron#15094.

@hossameldeen
Copy link
Author

Your work on localshortcut wasn't in vain indeed, @pfrazee :-)

@gabefair
Copy link

gabefair commented Oct 21, 2019

I am also encountering this issue. Using the keyboard shortcut ctrl+F on Windows 10-Pro (1903) 64-bit.
My logs are below:

beaker Uncaught exception: Error: Must provide a non-empty search content
at WebContents.findInPage (C:\Users\user\AppData\Local\Programs\beaker-browser\resources\electron.asar\browser\api\web-contents.js:321:15)
at View.moveInpageFind (C:\Users\user\AppData\Local\Programs\beaker-browser\resources\app.asar\background-process.build.js:2494:22)
at View.showInpageFind (C:\Users\user\AppData\Local\Programs\beaker-browser\resources\app.asar\background-process.build.js:2471:12)
at click (C:\Users\user\AppData\Local\Programs\beaker-browser\resources\app.asar\background-process.build.js:1847:26)
at MenuItem.click (C:\Users\user\AppData\Local\Programs\beaker-browser\resources\electron.asar\browser\api\menu-item.js:56:9)
at Function.executeCommand (C:\Users\user\AppData\Local\Programs\beaker-browser\resources\electron.asar\browser\api\menu.js:31:13)

@pfrazee pfrazee closed this as completed May 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants