Skip to content

Open a new window with Open VS Code Desktop #5703

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

Closed
sharkymark opened this issue Jan 12, 2023 · 4 comments · Fixed by coder/vscode-coder#52
Closed

Open a new window with Open VS Code Desktop #5703

sharkymark opened this issue Jan 12, 2023 · 4 comments · Fixed by coder/vscode-coder#52
Assignees

Comments

@sharkymark
Copy link
Contributor

The awesome new feature, Open VS Code Desktop, opens the existing local VS Code window so when I close it, I have to re-open my local 'workspace' again.

I suggest the default behavior be to open a new VS Code window locally.

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 23, 2023

I believe the URI that handles the opening in VS Code is here:

const query = new URLSearchParams({
owner: userName,
workspace: workspaceName,
url: location.origin,
token: key,
})
if (agentName) {
query.set("agent", agentName)
}
location.href = `vscode://coder.coder-remote/open?${query.toString()}`

So we'd have to see what params vscode:// and hope that it supports something like newWindow=true or something.

Looks like these are our query params so we could support it on this side. Curious though if we can support it from the vscode-coder extension side.

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 23, 2023

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 23, 2023

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 23, 2023

Ah maybe this is it? https://github.com/coder/vscode-coder/blob/ec4aa92aa7717740a5a1a66d4350236c9bcf913e/src/commands.ts#L200

see reuseWindow: true. We'd want to set that to false.

So seems like it's possible if I'm following the logic correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants