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

Don't trigger backspace global keybind when in text fields #85

Closed
AndreVolpi opened this issue Jul 30, 2024 · 5 comments · Fixed by #91
Closed

Don't trigger backspace global keybind when in text fields #85

AndreVolpi opened this issue Jul 30, 2024 · 5 comments · Fixed by #91

Comments

@AndreVolpi
Copy link

Hello, when I am querying tables, sometimes I make typos and my immediate response is to use backspace, but then it moves me to connection screen, would be nice if in editor fields backspace worked like a backspace.

@jorgerojas26
Copy link
Owner

Backspace should work properly when you are in an input, like the "WHERE" input and the SQL Editor. I cannot reproduce the problem.

@joelpaulkoch
Copy link

Hi, thanks for creating lazysql, this looks cool and fun. I have the same problem. I'm on MacOS and lazysql version 0.2.4

@joelpaulkoch
Copy link

joelpaulkoch commented Aug 13, 2024

I can actually reproduce the problem on my linux machine (which says version "dev", but that should be version 0.2.4 as well, from nixpkgs unstable), it occurs when following these steps:

  1. open a connection, go into the sql editor (ctrl+e), now if I type something and press backspace everything is still fine
  2. leave the sql editor (I press ESC)
  3. go back into the sql editor (I click inside)
  4. the previous text is still there, if I press backspace I'm back to the connection screen

I'm not sure if this is the same issue but this way I can trigger it.

@97albertus
Copy link

I'm experiencing this too (MacOS Sonoma), found a temporary fix for this is using ctrl+backspace instead of just backspace

@sktzofrenic
Copy link

I have the same problem with the backspace key.

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

Successfully merging a pull request may close this issue.

5 participants