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

Splitting terminal ignores "terminal.integrated.splitCwd": "inherited" and instead starts in workspace root #117079

Closed
chrisdakin opened this issue Feb 19, 2021 · 7 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label
Milestone

Comments

@chrisdakin
Copy link

chrisdakin commented Feb 19, 2021

  • VS Code Version: 1.53.2
    image

  • OS Version: 11.2.1
    image

Steps to Reproduce:

  1. Make sure terminal.integrated.splitCwd is set to inherited
  2. Change directory
  3. Split terminal (hotkey or by clicking split terminal)

Expected: New terminal starts in previous terminal's directory
Actual: New terminal starts in workspace directory

Screen.Recording.2021-02-19.at.2.06.55.PM.mov

Does this issue occur when all extensions are disabled?: Yes

Additional notes: This happens 100% of the time for me. It first started happening when I updated both to the newest version of VS Code (January) and to Big Sur (updated both at the same time). This happens when the default integrated terminal is bash and when it is zsh.

@meganrogge meganrogge added terminal General terminal issues that don't fall under another label bug Issue identified by VS Code Team member as probable bug labels Feb 26, 2021
@meganrogge meganrogge added this to the Backlog milestone Feb 26, 2021
@Tyriar
Copy link
Member

Tyriar commented Feb 26, 2021

This was disabled on Big Sur only as a workaround for #105446

@Tyriar
Copy link
Member

Tyriar commented Feb 26, 2021

I see it was reverted upstream, we will eventually enable it again when it's definitely fixed in Electron @deepak1556 ?

@deepak1556
Copy link
Collaborator

deepak1556 commented Feb 26, 2021

@Tyriar the libuv fix is not reverted in vscode, I haven't responded to the last comment in #105446, tl:dr; I have evaluated the failing scenarios in the libuv case and added workarounds for them in internal builds.

@sguillia
Copy link

@Tyriar This regression just made it through 1.61.0 and also reproduces on Linux

1.60.2

Expected and actual: New terminal starts in previous terminal's directory

1.61.0

Expected: New terminal starts in previous terminal's directory
Actual: New terminal starts in workspace directory


I've been used to the previous behavior for years, would it be possible to roll back?

Working version

Version: 1.60.2
Commit: 7f6ab5485bbc008386c4386d08766667e155244e
Date: 2021-09-22T12:01:43.795Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 4.15.0-142-generic

Non-working version

Version: 1.61.0
Commit: ee8c7def80afc00dd6e593ef12f37756d8f504ea
Date: 2021-10-07T18:11:02.929Z
Electron: 13.5.1
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 4.15.0-142-generic

@Tyriar
Copy link
Member

Tyriar commented Oct 11, 2021

@meganrogge could the event change in 6a7445b have caused that breakage on Linux?

@ebeloded
Copy link

ebeloded commented Nov 5, 2021

With every VSCode I hope this issue is fixed. In the latest release - it still isn't. I'm experiencing this on Mac.

@Tyriar
Copy link
Member

Tyriar commented Dec 12, 2022

Splitting is working fine on the latest macOS now. Please open a new issue with specific repro steps if you see this issue again.

@Tyriar Tyriar closed this as completed Dec 12, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jan 27, 2023
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 terminal General terminal issues that don't fall under another label
Projects
None yet
Development

No branches or pull requests

6 participants