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

Running command in terminal cannot be canceled #13560

Closed
Tracked by #13192
sgraband opened this issue Apr 3, 2024 · 3 comments · Fixed by #13564
Closed
Tracked by #13192

Running command in terminal cannot be canceled #13560

sgraband opened this issue Apr 3, 2024 · 3 comments · Fixed by #13564
Milestone

Comments

@sgraband
Copy link
Contributor

sgraband commented Apr 3, 2024

Bug Description:

Currently it seems like ctrl-c is not properly forwarded/registered on the built-in terminal. Pressing the keys does not cancel the currently running command.

Steps to Reproduce:

  1. Open a terminal
  2. Run a long-running command (e.g. yarn start)
  3. Try to cancel the command with ctrl-c
  4. Observe that the command i s not cancelled.

Additional Information

  • Operating System: Ubuntu 22.04
  • Theia Version: Theia IDE 1.48.0 (preview)
@tsmaeder
Copy link
Contributor

tsmaeder commented Apr 3, 2024

@tsmaeder
Copy link
Contributor

tsmaeder commented Apr 3, 2024

Turns out, this has nothing to do with the above change. There are several keybindings for ctrl-c the keybindings are traversed in registration order (per scope) and the first one that matches and is enabled is returned. In this case, it's copy (set core/browser/keybinding.ts#matchKeybinding(). If this ever worked, it's by accident.

@JonasHelming JonasHelming mentioned this issue Apr 4, 2024
55 tasks
@tsmaeder
Copy link
Contributor

tsmaeder commented Apr 4, 2024

While the above statement is true, the change in behavior is due to a different commit: https://github.com/eclipse-theia/theia/pull/13497/files#diff-549a2a9faed260a01fce0f2c2c3f3c4167d3fed2181fcd0bbc7a39b996082437R500

The "passthrough" pseudo-command has no handler and therefore is never enabled.

tsmaeder added a commit to tsmaeder/theia that referenced this issue Apr 4, 2024
Fixes eclipse-theia#13560

Contributed on behalf of STMicroelectronics

Signed-off-by: Thomas Mäder <t.s.maeder@gmail.com>
tsmaeder added a commit that referenced this issue Apr 4, 2024
…13564)

Fixes #13560

Contributed on behalf of STMicroelectronics

Signed-off-by: Thomas Mäder <t.s.maeder@gmail.com>
jfaltermeier pushed a commit that referenced this issue Apr 5, 2024
…13564)

Fixes #13560

Contributed on behalf of STMicroelectronics

Signed-off-by: Thomas Mäder <t.s.maeder@gmail.com>
@jfaltermeier jfaltermeier added this to the 1.49.0 milestone Apr 29, 2024
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.

3 participants