Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Dynamically update Open in Terminal explorer context menu item based on configuration #99542

Closed
Tyriar opened this issue Jun 6, 2020 · 1 comment · Fixed by #99543
Closed
Assignees
Labels
polish Cleanup and polish issue terminal General terminal issues that don't fall under another label
Milestone

Comments

@Tyriar
Copy link
Member

Tyriar commented Jun 6, 2020

Currently "Open in Terminal" is part of the core explorer context menu which is not very descriptive as it can be configured to open the external terminal as well. Based on terminal.external.*Exec/terminal.explorerKind we can update the name of the item to these:

@Tyriar Tyriar added polish Cleanup and polish issue terminal General terminal issues that don't fall under another label labels Jun 6, 2020
@Tyriar Tyriar added this to the June 2020 milestone Jun 6, 2020
@Tyriar Tyriar self-assigned this Jun 6, 2020
Tyriar added a commit that referenced this issue Jun 7, 2020
Tyriar added a commit that referenced this issue Jun 7, 2020
@Tyriar
Copy link
Member Author

Tyriar commented Jun 29, 2020

Verifier: test the context button on Windows 10 with the following settings to test the 3 different supported labels:

  • "terminal.explorerKind": "external"
  • "terminal.external.windowsExec": "wt"

@github-actions github-actions bot locked and limited conversation to collaborators Jul 22, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
polish Cleanup and polish issue terminal General terminal issues that don't fall under another label
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant