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

Crashing helix in mouse mode leaves terminal in bad state #2250

Closed
Zoybean opened this issue Apr 24, 2022 · 2 comments
Closed

Crashing helix in mouse mode leaves terminal in bad state #2250

Zoybean opened this issue Apr 24, 2022 · 2 comments
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug

Comments

@Zoybean
Copy link
Contributor

Zoybean commented Apr 24, 2022

Summary

While mouse was enabled, crashed helix.
In the terminal afterward, moving the mouse caused a spam of keycodes to the terminal
image

Reproduction Steps

I tried this:

  1. Set mouse mode in config.toml
  2. Start helix
  3. Crash helix eg using kill

I expected this to happen:

  • Terminal is left in the same state as before running helix

Instead, this happened:

  • Terminal is left in a state where it responds to mouse movements by typing junk

Helix log

No response

Platform

Linux (Ubuntu on WSL)

Terminal Emulator

Windows Terminal Version: 1.12.10982.0

Helix Version

22.03-121-g6de2e763

@Zoybean Zoybean added the C-bug Category: This is a bug label Apr 24, 2022
@the-mikedavis the-mikedavis added the A-helix-term Area: Helix term improvements label Apr 24, 2022
@eduardorittner
Copy link
Contributor

Shouldn't this issue be closed due to #7236?

@the-mikedavis
Copy link
Member

Yeah we can say this was mostly fixed by #7236 - we now intercept kill signals and close gracefully. For the other part of this (crashes) I'm not sure we can do much better - we have a panic handler that tries to restore the terminal but it doesn't have access to program state so it can only really guess how to restore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug
Projects
None yet
Development

No branches or pull requests

3 participants