Closed navincodesalot closed 2 years ago
This will break users on older versions of vscode, shouldn't the fix test the paths, then pick the correct var based on which path exists?
This will break users on older versions of vscode, shouldn't the fix test the paths, then pick the correct var based on which path exists?
So then could we provide this as a different version?
Ye. Ill work on testing the paths
This will break users on older versions of vscode, shouldn't the fix test the paths, then pick the correct var based on which path exists?
How would you think we could go about verifying what version of vs they are using? Only VSCode Insiders has this fix
Can we get this PR merged? The latest VSCode update needs this fix.
bump
bump
@robb0wen we hope to merge it
I couldn't merge this as-is because, as @pulberg pointed out, it would break the extension for older versions of VS code. I've made a tweak to the PR to run a check for the user's current VS code version which should prevent any issues.
I've tested this as working on Mac. Could someone running windows please check that this latest update works? Once we're happy it is, I'll merge this in
I couldn't merge this as-is because, as @pulberg pointed out, it would break the extension for older versions of VS code. I've made a tweak to the PR to run a check for the user's current VS code version which should prevent any issues.
I've tested this as working on Mac. Could someone running windows please check that this latest update works? Once we're happy it is, I'll merge this in
Where is this? I can test just cant find lmao
@navincodesalot It's in the commit history for the PR: https://github.com/robb0wen/synthwave-vscode/pull/264/commits/9f05676efa92763d0be8f06f3475cb20a1283e15
Here is a direct link to the file that you would replace your extension.js with:
@robb0wen Windows11 user here, seemed to work great!
I couldn't merge this as-is because, as @pulberg pointed out, it would break the extension for older versions of VS code. I've made a tweak to the PR to run a check for the user's current VS code version which should prevent any issues.
I've tested this as working on Mac. Could someone running windows please check that this latest update works? Once we're happy it is, I'll merge this in
electron-sandbox
and it's files was added in Aug 2020 first. If we have to cover the older version of VSCode, checking the existance of electron-sandbox
might not be right way.
I think checking sandbox-only by this would be better.
const browserWorkbenchPath = base + (isWin ? "\\electron-browser\\workbench" : "/electron-browser/workbench");
const usesSandbox = !fs.existsSync(browserWorkbenchPath);
Hey Guys, for mac users, this is what I did:
Hey Guys, for mac users, this is what I did:
- Open your terminal, and type the following: "cd .vscode/extensions"
- Then type "ls" to get a list view of all the files. you are looking for this file: "robbowen.synthwave-vscode-0.1.11"
- Once you confirm you have this file, type the following: "cd robbowen.synthwave-vscode-0.1.11"
- Type: "code ." Wait about 20seconds, and VS code will appear.
- Look in the src folder, and find the extension.js file. Open it.
- Delete everything in this file, and replace with everything in this link: "https://raw.githubusercontent.com/robb0wen/synthwave-vscode/9f05676efa92763d0be8f06f3475cb20a1283e15/src/extension.js"
- Save file, and quit VS code.
- Open VS code again, with whatever project you are working on. Reactivate Synthwave '84, Cmd Shift P + Enable Neon Dreams.
- Done
- You're Welcome
win11 with latest(Aug 2022) vscode works~
@mcpads You're absolute right. I missed that it had been in previous versions.
There's another PR which fixes the issue and includes a better, version-based check for the electron version. Simply for ease of it all being in one place, I'll merge that one and close this PR.
Thanks all for your input and testing on this one
As shown in #263 the fix by @mcpads is fixed with the addition of
electron-sandbox
instead ofelectron-browser
.electron-browser
was removed in commit https://github.com/microsoft/vscode/tree/f4f1b04d872a2b94d9a5105a1eefb81a213c07f2 and was renamed.These lines need to be changed: https://github.com/robb0wen/synthwave-vscode/blob/07273cd1e1fb6d0873dd96ab9f2e291eb136bee5/src/extension.js#L30-L40
https://github.com/robb0wen/synthwave-vscode/blob/07273cd1e1fb6d0873dd96ab9f2e291eb136bee5/src/extension.js#L109-L113