OCram85/code-server
Archived
1
0
Fork 0
Code Pull Requests 2 Activity

fix: use correct command to Open Folder on Welcome page (#3437)

Browse Source 
* chore: update CHANGELOG

* fix: use openFolder in welcomePage

In 1.56.2, they modified the WelcomePage use of OpenFileFolder in
src/vs/workbench/contrib/debug/browser/welcomeView.ts to only use on macOS +
Desktop (i.e. not Web). This caused the "Open Folder" to not work on macOS +
macOS-like environments.

Instead, we use OpenFolder which is a registered command and works as expected.
This commit is contained in:
Joe Previte 2021-05-21 12:27:50 -07:00 committed by GitHub
parent ac965176ca
commit 604491d333
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGELOG.md
View File

@ -71,6 +71,7 @@ VS Code v1.56.1
- fix: Check the logged user instead of $USER #3330 @videlanicolas - fix: Check the logged user instead of $USER #3330 @videlanicolas
- fix: Fix broken node_modules.asar symlink in npm package #3355 @code-asher - fix: Fix broken node_modules.asar symlink in npm package #3355 @code-asher
- fix: Update cloud agent to fix version issue #3342 @oxy - fix: Update cloud agent to fix version issue #3342 @oxy
- fix: use correct command to Open Folder on Welcome page #3437 @jsjoeio
### Documentation ### Documentation

2
lib/vscode/src/vs/workbench/contrib/welcome/page/browser/vs_code_welcome_page.ts
View File

@ -20,7 +20,7 @@ export default () => `
<h2 class="caption">${escape(localize('welcomePage.start', "Start"))}</h2> <h2 class="caption">${escape(localize('welcomePage.start', "Start"))}</h2>
<ul> <ul>
<li><a href="command:workbench.action.files.newUntitledFile">${escape(localize('welcomePage.newFile', "New file"))}</a></li> <li><a href="command:workbench.action.files.newUntitledFile">${escape(localize('welcomePage.newFile', "New file"))}</a></li>
<li class="mac-only"><a href="command:workbench.action.files.openFileFolder">${escape(localize('welcomePage.openFolder', "Open folder..."))}</a> or <a href="command:git.clone">${escape(localize('welcomePage.gitClone', "clone repository..."))}</a></li> <li class="mac-only"><a href="command:workbench.action.files.openFolder">${escape(localize('welcomePage.openFolder', "Open folder..."))}</a> or <a href="command:git.clone">${escape(localize('welcomePage.gitClone', "clone repository..."))}</a></li>
<li class="windows-only linux-only"><a href="command:workbench.action.files.openFolder">${escape(localize('welcomePage.openFolder', "Open folder..."))}</a> or <a href="command:git.clone">${escape(localize('welcomePage.gitClone', "clone repository..."))}</a></li> <li class="windows-only linux-only"><a href="command:workbench.action.files.openFolder">${escape(localize('welcomePage.openFolder', "Open folder..."))}</a> or <a href="command:git.clone">${escape(localize('welcomePage.gitClone', "clone repository..."))}</a></li>
</ul> </ul>
</div> </div>