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

workspace::NewTerminal opens two terminal instead of one #4567

Closed
1 task done
networkhermit opened this issue Jul 31, 2023 · 3 comments · Fixed by #10721
Closed
1 task done

workspace::NewTerminal opens two terminal instead of one #4567

networkhermit opened this issue Jul 31, 2023 · 3 comments · Fixed by #10721
Assignees
Labels
defect [core label] terminal Feedback for terminal integration, shell commands, etc

Comments

@networkhermit
Copy link

Check for existing issues

  • Completed

Describe the bug / provide steps to reproduce it

In keymap and command palette, workspace::NewTerminal opens two terminal instead of one. The default key binding ctrl-` is fine though.

Environment

Zed: v0.96.4 (stable)
OS: macOS 13.4.1
Memory: 16 GiB
Architecture: aarch64

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

No response

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.

If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.

No response

@networkhermit networkhermit added admin read Pending admin review defect [core label] triage Maintainer needs to classify the issue labels Jul 31, 2023
@JosephTLyons JosephTLyons added terminal Feedback for terminal integration, shell commands, etc and removed triage Maintainer needs to classify the issue admin read Pending admin review labels Aug 3, 2023
@JosephTLyons
Copy link
Contributor

Ah yeah, its happening because opening the terminal dock, when no terminals exist, automatically adds one, and then the action is adding another. Nice catch @networkhermit.

@JosephTLyons JosephTLyons transferred this issue from zed-industries/community Jan 24, 2024
@networkhermit
Copy link
Author

@JosephTLyons

hello, is there any update about this issue?

The Open in Terminal action on folders in the project panel would also opens two terminal instances.

Besides, is it possible to add a custom keybinding to open the directory of the current buffer in terminal? I've tried project_panel::OpenInTerminal but got no luck (#4566):

  {
    "bindings": {
      "shift-k": "project_panel::OpenInTerminal"
    },
    "context": "Editor && vim_mode == normal && !VimWaiting && !menu"
  }

@networkhermit
Copy link
Author

@JosephTLyons

hello, is there any update about this issue?

The Open in Terminal action on folders in the project panel would also opens two terminal instances.

Besides, is it possible to add a custom keybinding to open the directory of the current buffer in terminal? I've tried project_panel::OpenInTerminal but got no luck (#4566):

  {
    "bindings": {
      "shift-k": "project_panel::OpenInTerminal"
    },
    "context": "Editor && vim_mode == normal && !VimWaiting && !menu"
  }

@ConradIrwin hi, could you help binging this issue and #4566 to the roadmap? It's not something urgent, but it bugs me for a long time and I have to fall back to vscode for even simple usage.

@SomeoneToIgnore SomeoneToIgnore self-assigned this Apr 18, 2024
SomeoneToIgnore added a commit that referenced this issue Apr 18, 2024
Fixes #4567

Release Notes:

- Fixed multiple terminals being opened on `workspace::NewTerminal`
calls ([4567](#4567))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect [core label] terminal Feedback for terminal integration, shell commands, etc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants