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

Terminal tab item doesn't seem to do anything #125045

Closed
jrieken opened this issue Jun 1, 2021 · 10 comments
Closed

Terminal tab item doesn't seem to do anything #125045

jrieken opened this issue Jun 1, 2021 · 10 comments
Assignees
Labels
insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue terminal General terminal issues that don't fall under another label terminal-tabs
Milestone

Comments

@jrieken
Copy link
Member

jrieken commented Jun 1, 2021

Testing #124716

  • have a single terminal
  • the terminal is rendered in the panel's toolbar
  • 🐛 the terminal is render as command (hover feedback, cursor style) but doesn't seem to do anything

Screen Shot 2021-06-01 at 11 00 30

@stevencl
Copy link
Member

stevencl commented Jun 1, 2021

I saw the same. I expected clicking that button to add another terminal, since alt-clicking it adds a new terminal and splits the two terminals. Maybe it shouldn't be clickable? Does it need to be shown when there is only one terminal?

@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2021

Left click used to open the dropdown menu, now it focuses the terminal to align more with how the normal tabs behave. Supporting the dropdown menu on left click was also overly complicated. Alt+click and middle+click also match the behavior of the regular tabs.

@Tyriar Tyriar added under-discussion Issue is under discussion for relevance, priority, approach terminal General terminal issues that don't fall under another label terminal-tabs labels Jun 1, 2021
@jrieken
Copy link
Member Author

jrieken commented Jun 1, 2021

now it focuses the terminal to align more with how the normal tabs behave.

But it's not a tab, it's a button...

@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2021

@misolori any thoughts?

@miguelsolorio
Copy link
Contributor

I do agree that having the hover feedback is odd when left click does nothing. So I think we have to make it clickable and either show a menu or create another terminal. Alternatively we could show the single tab in the list but we already went that route. Disabling it would not help as there are multiple actions you can do on a single terminal.

@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2021

@misolori left click does focus the terminal, it's just not very noticeable (or useful?), this aligns with single or double left click in the full tab view (depending on focusMode). I can bring the menu back on left click though.

@jrieken
Copy link
Member Author

jrieken commented Jun 1, 2021

...alternatively we could show the single tab in the list but we already went that route.

😢 I still miss that...

@miguelsolorio
Copy link
Contributor

@Tyriar yea that is too subtle and not very useful since you only have a single terminal so the view doesn't change.

@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2021

@jrieken if you want that:

"terminal.integrated.tabs.hideCondition": "never",
"terminal.integrated.tabs.showActiveTerminal": "never"

After feedback from the team we went with a more lightweight single terminal approach.

@Tyriar Tyriar added polish Cleanup and polish issue and removed under-discussion Issue is under discussion for relevance, priority, approach labels Jun 1, 2021
@Tyriar Tyriar added this to the May 2021 milestone Jun 1, 2021
@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2021

Action item: Make left click open the menu again

@github-actions github-actions bot locked and limited conversation to collaborators Jul 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue terminal General terminal issues that don't fall under another label terminal-tabs
Projects
None yet
Development

No branches or pull requests

6 participants
@stevencl @jrieken @Tyriar @meganrogge @miguelsolorio and others