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

switch terminal action breaks (and creates...?) terminal editors when switching between terminal editors in different groups #127901

Closed
meganrogge opened this issue Jul 2, 2021 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders terminal General terminal issues that don't fall under another label terminal-editors verified Verification succeeded
Milestone

Comments

@meganrogge
Copy link
Contributor

meganrogge commented Jul 2, 2021

Open latest insiders
Create new terminal editor
Split it using editor split button on top right.
Switch from one terminal to the other (several times) using CMD+P term listing.
I hope that my message is clear enough. If not don't hesitate to tell me, and I'll do a video for you on Monday.

from #10546 (comment)

@meganrogge meganrogge added bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label terminal-editors labels Jul 2, 2021
@meganrogge meganrogge modified the milestones: June 2021, July 2021 Jul 2, 2021
@meganrogge meganrogge changed the title switch terminal action breaks (and creates...?) terminal editors switch terminal action breaks (and creates...?) terminal editors when switching between terminal editors in different groups Jul 2, 2021
meganrogge added a commit that referenced this issue Jul 4, 2021
@meganrogge meganrogge modified the milestones: July 2021, June 2021 Jul 4, 2021
@meganrogge meganrogge added the candidate Issue identified as probable candidate for fixing in the next release label Jul 4, 2021
@rzhao271 rzhao271 added verified Verification succeeded verification-found Issue verification failed and removed verified Verification succeeded labels Jul 7, 2021
@rzhao271
Copy link
Contributor

rzhao271 commented Jul 7, 2021

Switching from terminal A to terminal B (once) results in terminal A's group picking up a new terminal, and terminal B breaking.

@rzhao271 rzhao271 reopened this Jul 7, 2021
@Tyriar
Copy link
Member

Tyriar commented Jul 7, 2021

Can repro:

image

It actually breaks the left side but only because it gets moved to the right side. I suspect this is related to the recent SIDE_GROUP change. I don't think this meets the bar to push the release back as it can only happen when changing groups via ctrl/cmd+p and the fragility of that code.

It can happen the other way too:

image

@Tyriar Tyriar modified the milestones: June 2021, July 2021 Jul 7, 2021
@chrmarti chrmarti removed the candidate Issue identified as probable candidate for fixing in the next release label Jul 7, 2021
@meganrogge
Copy link
Contributor Author

FWIW, the SIDE_GROUP change was reverted.

@Tyriar Tyriar modified the milestones: July 2021, August 2021 Jul 26, 2021
@meganrogge meganrogge removed the verification-found Issue verification failed label Jul 28, 2021
meganrogge added a commit that referenced this issue Aug 5, 2021
@mjbvz mjbvz added the verified Verification succeeded label Aug 25, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Sep 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders terminal General terminal issues that don't fall under another label terminal-editors verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants
@Tyriar @rzhao271 @chrmarti @mjbvz @meganrogge and others