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

[regression] workbench.files.action.showActiveFileInExplorer broken #41097

Closed
jrieken opened this issue Jan 3, 2018 · 4 comments
Closed

[regression] workbench.files.action.showActiveFileInExplorer broken #41097

jrieken opened this issue Jan 3, 2018 · 4 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug file-explorer Explorer widget issues verified Verification succeeded
Milestone

Comments

@jrieken
Copy link
Member

jrieken commented Jan 3, 2018

{
		"key": "shift+cmd+e",
		"command": "workbench.files.action.showActiveFileInExplorer"
	},

I have the above snippet in my keybinding.json-file and it allowed me to reveal the current file in the explorer (since I don't do that automatically). This is not working anymore.

@vscodebot vscodebot bot added workbench workbench-tabs VS Code editor tab issues labels Jan 3, 2018
@jrieken jrieken changed the title workbench.files.action.showActiveFileInExplorer broken [regression] workbench.files.action.showActiveFileInExplorer broken Jan 3, 2018
@bpasero bpasero assigned isidorn and unassigned bpasero Jan 4, 2018
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug file-explorer Explorer widget issues and removed workbench workbench-tabs VS Code editor tab issues labels Jan 4, 2018
@bpasero
Copy link
Member

bpasero commented Jan 4, 2018

@isidorn could be a regression from your commands work?

@isidorn isidorn added this to the December 2017/January 2018 milestone Jan 4, 2018
@isidorn
Copy link
Contributor

isidorn commented Jan 4, 2018

I just tried this with latest insiders on the mac and it works fine for me.
@bpasero can you also reproduce this? I checked my PR and everything looks fine around this comand
@jrieken do you have some other keybindings? Was this working yesterday and stopped working today? Does the command work if you trigger it via command palette?

@isidorn isidorn added the info-needed Issue requires more information from poster label Jan 4, 2018
@jrieken
Copy link
Member Author

jrieken commented Jan 4, 2018

@isidorn These are all keybindings I have defined

// my keys
[
	{
		"key": "shift+cmd+[",
		"command": "workbench.action.openView"
	},
	{
		"key": "cmd+r",
		"command": "workbench.action.reloadWindow"
	},
	{
		"key": "alt+cmd+i",
		"command": "workbench.action.toggleDevTools"
	},
	{
		"key": "shift+cmd+i",
		"command": "_webview.openDevTools"
	},
	{
		"key": "cmd+k j",
		"command": "_generateCommandsDocumentation"
	},
	{
		"key": "shift+cmd+e",
		"command": "workbench.files.action.showActiveFileInExplorer"
	},
	{
		"key": "shift+cmd+e",
		"command": "workbench.view.explorer",
		"when": "!editorIsOpen"
	},
	{
		"key": "cmd+l c",
		"command": "extension.copyGitHubLinkToClipboard"
	},
	{
		"key": "cmd+w",
		"command": "workbench.action.closeEditorsInGroup",
		"args": null
	},
	{
		"key": "cmd+e",
		"command": "workbench.action.quickOpen"
	},
	{
		"key": "cmd+e",
		"command": "workbench.action.quickOpenNavigateNext",
		"when": "inQuickOpen"
	},
	{
		"key": "shift+cmd+e",
		"command": "workbench.action.quickOpenNavigatePrevious",
		"when": "inQuickOpen"
	},
	{
		"key": "cmd+k f",
		"command": "editor.action.format",
		"when": "editorTextFocus"
	},
	{
		"key": "cmd+k",
		"command": "workbench.action.terminal.clear",
		"when": "terminalFocus"
	},
	{
		"key": "f9",
		"command": "-sortLines.sortLines",
		"when": "editorTextFocus"
	},
	{
		"key": "ctrl+w",
		"command": "expand_region",
		"when": "editorTextFocus"
	},
	{
		"key": "ctrl+shift+w",
		"command": "undo_expand_region",
		"when": "editorTextFocus && editorHasSelection"
	},
	{
		"key": "cmd+k 1",
		"command": "editor.action.insertSnippet",
		"when": "editorTextFocus",
		"args": {
			"snippet": "console.log('${1:Here-${TM_LINE_INDEX}$2}')$0"
		}
	},
	{
		"key": "shift+escape",
		"command": "acceptSnippet"
	},
	{
		"key": "cmd+k ,",
		"command": "workbench.action.openRawDefaultSettings"
	}
]

@jrieken jrieken removed the info-needed Issue requires more information from poster label Jan 4, 2018
@isidorn isidorn closed this as completed in 0e2f96f Jan 4, 2018
@isidorn
Copy link
Contributor

isidorn commented Jan 4, 2018

I introduced this regression with the PR.
@jrieken thanks for finding this

@bpasero bpasero added the verified Verification succeeded label Feb 1, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Feb 18, 2018
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 file-explorer Explorer widget issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants