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

When shift is pressed, the string shift is printed #20963

Closed
1 task done
jizhi0v0 opened this issue Nov 21, 2024 · 17 comments
Closed
1 task done

When shift is pressed, the string shift is printed #20963

jizhi0v0 opened this issue Nov 21, 2024 · 17 comments
Labels
duplicate Additional reports of an existing issue

Comments

@jizhi0v0
Copy link

Check for existing issues

  • Completed

Describe the bug / provide steps to reproduce it

when knock the shift key, the string shift is append in current line

CleanShot.2024-11-21.at.11.47.06.mp4

Environment

zed preview 0.162.3 version
Apple M3 Max
36 GB
15.1 (24B83)

If applicable, add mockups / screenshots to help explain present your vision of the feature

No response

If applicable, attach your Zed.log file to this issue.

Zed.log
[Zed.log](https://github.com/user-attachments/files/17839514/Zed.log)

@jizhi0v0 jizhi0v0 added admin read Pending admin review bug [core label] triage Maintainer needs to classify the issue labels Nov 21, 2024
@jizhi0v0
Copy link
Author

This issue only in 0.163.0 preview and 0.162.3 stable

@jizhi0v0
Copy link
Author

jizhi0v0 commented Nov 21, 2024

git bisect start
git bisect good v0.161.2
git bisect bad v0.162.3

Finally c4a382d is the first bad commit

@caturbgs
Copy link

This is very frustrating issue I got today after updating Zed, hope this will take up soon

@eatenpancreas
Copy link

Having the same issue here

@lebe-dev
Copy link

Approved, v0.161.2 is not affected

@piechologist
Copy link

Do you all have a key binding for "shift shift"?

For me, pressing shift twice runs the bound action as intended. Pressing shift just once inserts shift into the editor after a split second. Removing the key binding solves the issue.

The same issue occurs with the other modifiers e.g., binding "cmd cmd" inserts platform.

@BraisonCrece
Copy link

Do you all have a key binding for "shift shift"?

For me, pressing shift twice runs the bound action as intended. Pressing shift just once inserts shift into the editor after a split second. Removing the key binding solves the issue.

The same issue occurs with the other modifiers e.g., binding "cmd cmd" and inserts platform.

Yes, I have shift shift binding and disabling it fixes the problem 👍

@eatenpancreas
Copy link

@piechologist do you mean like this? I'm getting it but very inconsistently so
image

@eatenpancreas
Copy link

eatenpancreas commented Nov 21, 2024

I figured out what's causing it on my end (sort of).

For me, it's only happening if i select something, while selected press shift (just once) and then leave the selection by clicking somewhere else. It'll then just get pasted wherever i clicked to next.

Screen.Recording.2024-11-21.at.12.25.12.mov

@piechologist
Copy link

For me, it's only happening if i select something

I can't replicate that. It doesn't matter if there's a selection or not. With the shift shift binding commented out, nothing gets inserted.

@xplodwild
Copy link

xplodwild commented Nov 21, 2024

Same happens for me on Linux v0.162.3, with a "shift shift" keybinding enabled, pressing shift then an arrow key leads to "shift" being inserted in the current buffer.

Disabling/commenting my "shift shift" keybinding, the problem goes away immediately.

@blueambr
Copy link

I don't have any bindings and this happens. It's a ridiculously annoying bug. Wonder how it was introduced into the stable build.

@HarshNarayanJha
Copy link
Contributor

Can confirm. It is happening on the latest stable. I do have a shift-shift key binding to open ctrl+p menu

System Info

Zed: v0.162.3 (Zed)
OS: Linux Wayland garuda unknown
Memory: 7.1 GiB
Architecture: x86_64
GPU: AMD Radeon Graphics (RADV RENOIR) || radv || Mesa 24.2.7-arch1.1

@VladimirTaytor
Copy link

Same here. I don't have "shift shift" custom bindings, but I do have "base_keymap": "JetBrains"
Please take a look asap, it's very annoying 🙏

@eatenpancreas
Copy link

Same here. I don't have "shift shift" custom bindings, but I do have "base_keymap": "JetBrains" Please take a look asap, it's very annoying 🙏

now that you say it, i also have JetBrains base keymap as well, so that could be a part of it

@modelga
Copy link

modelga commented Nov 21, 2024

Hi!, It's bothering me too, additionally, when I add an extra keybinding "cmd cmd": "workspace::ToggleZoom" I've got another text inserted in editor -> platform while I pressed 'cmd' only once and then clicked somewhere else. I have base_keymap derived from VSCode.

Using MacOS 14.7.1 on M1 arm64 architecture

@piechologist
Copy link

Zed Preview 0.163.1 is out with a fix.

Thanks to the Zed team for the quick response! 🚀

@notpeter notpeter added duplicate Additional reports of an existing issue and removed bug [core label] triage Maintainer needs to classify the issue admin read Pending admin review labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Additional reports of an existing issue
Projects
None yet
Development

No branches or pull requests