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

Double click of CTRL + F does not fallback to Chromium search bar #6930

Open
RafalSkolasinski opened this issue Aug 1, 2019 · 8 comments
Open
Labels
pkg:documentsearch status:Needs Discussion tag:Design and UX tag:Keyboard Compatibility issues with keyboards, including locale-specific and layout-specific issues
Milestone

Comments

@RafalSkolasinski
Copy link

Describe the bug
Double click of CTRL + F does not fallback to Chromium search bar.

To Reproduce

  1. Run jupyterlab with
docker run --rm -it -p 127.0.0.1:8888:8888 -e JUPYTER_ENABLE_LAB=yes jupyter/minimal-notebook:307ad2bb5fce
  1. Go to the jupyterlab and open new notebook
  2. With cursor in edit mode in a cell click CTRL+F twice.

Expected behavior
The browser's searchfield is the active one instead of jupyter lab built-in.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • Ubuntu 18.04
  • Browser [e.g. chrome 69.0, firefox 62.0]
  • JupyterLab [1.0.4] from jupyter/minimal-notebook:307ad2bb5fce docker image
@jasongrout
Copy link
Contributor

jasongrout commented Aug 1, 2019

This is expected behavior - Ctrl+F invokes the JupyterLab search, and repeated uses focus the JupyterLab search interface. Are you requesting this as a new feature?

@RafalSkolasinski
Copy link
Author

RafalSkolasinski commented Aug 1, 2019

I thought that two consecutive invokes of CTRL + F were suppose to fallback to browser's search bar (ref).

If that's not the case then indeed I would be requesting this as a new feature.

Or is there an option to just disable JupyterLab search bar?

@jasongrout
Copy link
Contributor

That reference was before we had an integrated search in JupyterLab.

You can disable the JupyterLab search on Ctrl+F by disabling that keyboard shortcut. Follow steps 1 and 2 of #6830

Then Ctrl+F invokes the codemirror search just for the cell. And doing Ctrl+F twice invokes the browser search.

@jasongrout
Copy link
Contributor

Tagging this as a feature request. Perhaps pressing Ctrl+F when the jlab search interface is focused should close the search and allow the browser search to appear? Or perhaps there should be an option for this?

@jasongrout
Copy link
Contributor

CC @aschlaep

@RafalSkolasinski
Copy link
Author

Perhaps pressing Ctrl+F when the jlab search interface is focused should close the search and allow the browser search to appear?

That sounds like a very good idea and goes along with what (some) people are used to.

@krassowski
Copy link
Member

krassowski commented Aug 5, 2019

Allow the browser search to appear - yes
Close the custom search - not until the jumping/performance issues are fully solved, please!

Rationale: it would be too easy to incidentally press a key sequence leading to a freeze of the interface when using large notebooks. As it does not seem easy to solve the performance issues, I would argue for caution when switching on/off the custom search functionality.

Also in another thread is has been mentioned that if you click on the menu bar, the browser search will appear (a workaround for current builds rather than a solution).

@sixers
Copy link

sixers commented May 13, 2020

Hijacking default browser keyboard shortcuts is extremely annoying, especially as common ones as search. There has to be a way to disable CodeMirror search. I can't even count how many times it made me pull my hair out.

Eventually, I was able to disable this in the browser itself (I'm using Firefox): https://superuser.com/a/1317514

Please add an option to disable shortcuts like this.

@JasonWeill JasonWeill added the tag:Keyboard Compatibility issues with keyboards, including locale-specific and layout-specific issues label Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:documentsearch status:Needs Discussion tag:Design and UX tag:Keyboard Compatibility issues with keyboards, including locale-specific and layout-specific issues
Projects
None yet
Development

No branches or pull requests

6 participants