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

Mac only: "Command F" vs "Command Shift F" #12421

Closed
zsiegel92 opened this issue Sep 28, 2020 · 2 comments · Fixed by #12422
Closed

Mac only: "Command F" vs "Command Shift F" #12421

zsiegel92 opened this issue Sep 28, 2020 · 2 comments · Fixed by #12422
Labels

Comments

@zsiegel92
Copy link

Attach (recommended) or Link to PDF file here:

Configuration:

  • Web browser and its version: Chrome 85.0.4183.121 (Official Build) (64-bit)
  • Operating system and its version: Mac 10.15.4
  • PDF.js version: 2.3.164
  • Is a browser extension: Yes

Steps to reproduce the problem:

  1. Open a PDf
  2. Press "Command + Shift + F"

What is the expected behavior? (add screenshot)

Chrome typically uses this shortcut for "Always Show Toolbar in Full Screen".

What went wrong? (add screenshot)

In PDF.js, this shortcut is captured by the extension as "Find". The shortcut "Command + F" is also captured and serves this purpose, so I don't believe "Command + Shift + F" should ALSO be captured for "Find". There is no way to configure the keyboard shortcuts captured by this extension in its options.

I prefer PDF.js to the built-in Chrome PDF viewer (mostly for its ability to open PDF documents to the previous location), but this makes it hard to use.

@zsiegel92
Copy link
Author

I am hoping this is not a very difficult issue to resolve. Please let me know whether this will be addressed. Thank you!

@timvandermeij
Copy link
Contributor

Fixed by the PR above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants