-
-
Notifications
You must be signed in to change notification settings - Fork 769
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
Strange keyboard input behaviour #4504
Comments
I have this problem too: What Operating System(s) are you seeing this problem on? Which Wayland compositor or X11 Window manager(s) are you using? WezTerm version Also experiencing this as fairly random but quite frequent. |
The worst is asking questions with default answer and unintentional ENTER... |
I tested wezterm on river as well - i.e https://github.com/riverwm/river. River uses wlroots 0.16 (while hyprland uses more or less HEAD of wlroots). I observed the same input behavior in river as in hyprland so this seems to be something that was introduced in wezterm relatively recently. I didn't have these issues in 20230712-072601-f4abf8fd. I can't use an older version of wezterm though as those won't launch on hyprland at least. |
When debugging key events (eg.
The timings in the log matches how it "feels". It's not immediate but, as the log indicates, the dupe is output about half a second later. Generally, when just writing text, I don't notice the problem as much as it doesn't seem to manifest if a key is pressed before about 5-600 ms has passed. |
@johnae Can you check it with recent commit? I don't see that behavior anymore. Could you confirm? |
Do you have any debug flag enabled? |
No. Any pointers? I mean I could build a dev build but what would I be looking for? If you mean log verbosity, I didn't set anything but can do that ofc if it'd be helpful. |
Nah... it randomly came back again. |
I suppose it could be related to that other issue. Still - I see this happening after a reboot with only a single terminal window and nothing else opened. That window was never not visible. I too ofc see the 100% cpu issue from that other issue. I'm not convinced they're related though. |
I believe I've found the likely issue. This became an issue when updating SCTK to 0.16.1 (from 0.16.0). This was done to support startup on newer compositors, so for most it'd be a big deal to downgrade both the compositor and wezterm. The issue is (likely) upstream in SCTK, more specifically this issue: Smithay/client-toolkit#356 - it fairly closely resembles the issue I'm seeing anyway. The fix commit is in SCTK 0.17.0 and 0.18.0 so basically wezterm would need to update to any of those versions, which may not be a small thing I suppose - I'm not too familiar with the wezterm code base yet. What are your thoughts on this @wez ? |
Elsewhere in wezterm's issue's tagged with wayland there is a discussion about the level of effort involved: the changes from 0.16 -> 0.17 and the wayland ecosystem crates are effectively a rewrite of wayland support in wezterm, for which I do not have time or energy. I would really welcome a PR for that! Alternatively, if sufficient funding could be provided, I could make time for this, but I view this as a large effort and no one has been serious about funding that effort so far. (https://github.com/sponsors/wez?frequency=one-time&sponsor=wez shows the ballpark for funding a large project). |
Yeah I get it. I'd actually welcome a challenge myself, I'm not a Rust beginner really. I think the main challenge would be wayland/smithay (haven't done much such work if any really). Well that and the constant absence of time. Regardless - wezterm is really awesome, thanks for making it! |
Doesn't seem to be an issue anymore. @johnae could you confirm? |
Not OP, but I'm still having this issue on current version of Wezterm + Wayland. |
@zach-wahrer stable? |
@grappas I'm running 20240203-110809-5046fc22. |
ok |
I'm closing this since the OP reports that their issue is resolved. If you are experiencing this or a similar issue, please try with the latest nightly build to confirm and then open a new issue of your own for that. |
It seems to be the case also. Thanks @wez |
What Operating System(s) are you seeing this problem on?
Linux Wayland
Which Wayland compositor or X11 Window manager(s) are you using?
Hyprland 0.31.0.r47.7f4b0aaa-1
WezTerm version
20231017-091526-fec90ae0
Did you try the latest nightly build to see if the issue is better (or worse!) than your current version?
Yes, and I updated the version box above to show the version of the nightly that I tried
Describe the bug
Sometimes keystrokes are registered double times after original single one. It appears after ≈500ms of original keystroke. This way comes up double spaces, double a's or worse - double new lines.
To Reproduce
Configuration
Expected Behavior
Keystrokes have to be registered as they should be.
Logs
15:41:05.005 INFO wezterm_gui > Spawned your command via the existing GUI instance. Use wezterm start --always-new-process if you do not want this behavior. Result=SpawnResponse { tab_id: 1, pane_id: 1, window_id: 1, size: TerminalSize { rows: 24, cols: 80, pixel_width: 640, pixel_height: 384, dpi: 0 } }
Anything else?
keystrokes.mp4
There was only one keystroke of "Return" in the beginning, only one keystroke of "d" in third prompt and one "w" in fourth.
The text was updated successfully, but these errors were encountered: