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

Start new tab in last used tab working dir path [BUG] #13430

Closed
QuAzI opened this issue Jul 5, 2022 · 6 comments
Closed

Start new tab in last used tab working dir path [BUG] #13430

QuAzI opened this issue Jul 5, 2022 · 6 comments
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.

Comments

@QuAzI
Copy link

QuAzI commented Jul 5, 2022

It is better to use last working dir for new tab instead of first random chosen path
image

@QuAzI QuAzI added the Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. label Jul 5, 2022
@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Jul 5, 2022
@QuAzI
Copy link
Author

QuAzI commented Jul 5, 2022

As example I started wt in any dir. Then I ran through FM (Explorer, TotalCommander... doesn't matter) CMD file which starts two copies of nodejs (es5, es6) in the same tab.

CMD example

start wt -d ./src/ApplicationFrontend/  pwsh.exe -NoExit -command npm run watch-debug-es5-only; split-pane -d ./src/ApplicationFrontend/ powershell.exe -NoExit -command npm run watch-debug-es6-only

now my script crashes because start path broken. It is not script path and not last used path. It means even if I run my script from wt - it crashes if wt started not from current path. So I must close all tabs and start from begin

[error 2147942667 (0x8007010b) when launching `pwsh.exe -NoExit -command npm run watch-debug-es5-only']
Could not access starting directory "./src/ApplicationFrontend/"

@QuAzI QuAzI changed the title Start new tab in last used tab working dir path Start new tab in last used tab working dir path [BUG] Jul 5, 2022
@QuAzI
Copy link
Author

QuAzI commented Jul 5, 2022

And even this workaround in start of script doesn't helps

cd %~dp0

@zadjii-msft
Copy link
Member

zadjii-msft commented Jul 7, 2022

Wait what exactly is the bug here?

Do you have "New instance behavior" set to "Create a new window" or "Attach to the most recent..."? Trying this out locally, I got the same error message when the tab was opening in an existing WT window. When it gets sent to a new window though, it seems to work fine

I was sure this was on the backlog somewhere... found it, or at least what I've been using to track this: #5506

@zadjii-msft zadjii-msft added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jul 8, 2022
@QuAzI
Copy link
Author

QuAzI commented Jul 11, 2022

"Attach to the most recent..." of course because it is more comfortable to have one window with multiple tabs. But it doesn't mean that this window combine only instances with same working dir.

@ghost ghost added Needs-Attention The core contributors need to come back around and look at this ASAP. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jul 11, 2022
@zadjii-msft
Copy link
Member

Okay, yea, that'll be fixed by #5506 then. Thanks!

/dup #5506

@zadjii-msft zadjii-msft closed this as not planned Won't fix, can't repro, duplicate, stale Jul 11, 2022
@ghost
Copy link

ghost commented Jul 11, 2022

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost ghost closed this as completed Jul 11, 2022
@ghost ghost added Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements Needs-Attention The core contributors need to come back around and look at this ASAP. labels Jul 11, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.
Projects
None yet
Development

No branches or pull requests

2 participants