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

"Reveal in Explorer" hotkey doesn't work #124825

Closed
oliversalzburg opened this issue May 28, 2021 · 8 comments
Closed

"Reveal in Explorer" hotkey doesn't work #124825

oliversalzburg opened this issue May 28, 2021 · 8 comments
Assignees
Labels
file-explorer Explorer widget issues *out-of-scope Posted issue is not in scope of VS Code under-discussion Issue is under discussion for relevance, priority, approach

Comments

@oliversalzburg
Copy link

Issue Type: Bug

Using the key combination (Shift+Alt+R) for "Reveal in Explorer" does nothing.

Using the context menu entry works as expected.

The troubleshooter output looks like it works:

[2021-05-28 12:12:37.602] [renderer1] [info] [KeybindingService]: \ From 2 keybinding entries, matched revealFileInOS, when: !editorFocus, source: built-in.
[2021-05-28 12:12:37.602] [renderer1] [info] [KeybindingService]: / Received  keydown event - modifiers: [shift,alt], code: KeyR, keyCode: 82, key: R
[2021-05-28 12:12:37.602] [renderer1] [info] [KeybindingService]: | Converted keydown event - modifiers: [shift,alt], code: KeyR, keyCode: 48 ('R')
[2021-05-28 12:12:37.602] [renderer1] [info] [KeybindingService]: | Resolving shift+alt+R
[2021-05-28 12:12:37.602] [renderer1] [info] [KeybindingService]: \ From 2 keybinding entries, matched revealFileInOS, when: !editorFocus, source: built-in.
[2021-05-28 12:12:37.769] [renderer1] [info] [KeybindingService]: + Storing single modifier for possible chord alt.
[2021-05-28 12:12:38.070] [renderer1] [info] [KeybindingService]: + Clearing single modifier due to 300ms elapsed.

VS Code version: Code 1.56.2 (054a929, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.19042
Remote OS version: Linux x64 5.4.72-microsoft-standard-WSL2
Fetching remote diagnostics for 'WSL: Ubuntu' failed: Canceled
Remote OS version: Linux x64 5.4.72-microsoft-standard-WSL2
Fetching remote diagnostics for 'WSL: Ubuntu' failed: Canceled

System Info
Item Value
CPUs Intel(R) Core(TM) i7-6850K CPU @ 3.60GHz (12 x 3592)
GPU Status 2d_canvas: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
oop_rasterization: enabled
opengl: enabled_on
rasterization: enabled
skia_renderer: enabled_on
video_decode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
Load (avg) undefined
Memory (System) 63.87GB (33.75GB free)
Process Argv --folder-uri=vscode-remote://wsl+Ubuntu/home/oliver/fairmanager/fairmanager/core-web3 --remote=wsl+Ubuntu --crash-reporter-id 05565d81-cfc4-40de-88df-1c479bbe1bcd
Screen Reader no
VM 0%
Item Value
Remote WSL: Ubuntu
OS Linux x64 5.4.72-microsoft-standard-WSL2
CPUs Intel(R) Core(TM) i7-6850K CPU @ 3.60GHz (12 x 3591)
Memory (System) 50.11GB (34.43GB free)
VM 0%

Fetching remote diagnostics for 'WSL: Ubuntu' failed: Canceled

Item Value
Remote WSL: Ubuntu
OS Linux x64 5.4.72-microsoft-standard-WSL2
CPUs Intel(R) Core(TM) i7-6850K CPU @ 3.60GHz (12 x 3591)
Memory (System) 50.11GB (34.43GB free)
VM 0%

Fetching remote diagnostics for 'WSL: Ubuntu' failed: Canceled

Extensions (27)
Extension Author (truncated) Version
javascript-ejs-support Dig 1.3.1
eslint-disable-snippets drK 1.3.0
tsl-problem-matcher eam 0.4.0
output-colorizer IBM 0.1.2
remote-containers ms- 0.177.2
remote-ssh ms- 0.65.4
remote-ssh-edit ms- 0.65.6
remote-wsl ms- 0.56.3
vscode-remote-extensionpack ms- 0.21.0
vscode-icons vsc 11.4.0
vscode-zipfs arc 2.3.0
better-toml bun 0.3.2
vscode-eslint dba 2.1.20
xml Dot 2.5.1
EditorConfig Edi 0.16.4
prettier-vscode esb 6.4.0
generateuuid Fir 0.0.5
shell-format fox 7.1.0
vscode-graphql Gra 0.3.16
terraform has 2.11.0
vscode-esformatter how 1.6.0
vscode-docker ms- 1.13.0
python ms- 2021.5.842923320
vscode-pylance ms- 2021.5.4
jupyter ms- 2021.6.832593372
vscode-paste-image mus 1.0.4
monolit Oli 0.0.33

(1 theme extensions excluded)

@vscodebot
Copy link

vscodebot bot commented May 28, 2021

@isidorn
Copy link
Contributor

isidorn commented May 28, 2021

@oliversalzburg Yeah that shortcut seems to work only when the focus is not in the editor. So if I put the focus in the explorer it works. The reason for this is that the find widget uses the same keybinding to toggle the regex.

You can always customise this in your keybinding preferences by changing the when clause.
I think this is behaving as designed, thus for now closing. If you disagree please let me know and we can reopen the issue

@isidorn isidorn closed this as completed May 28, 2021
@isidorn isidorn added *as-designed Described behavior is as designed file-explorer Explorer widget issues labels May 28, 2021
@oliversalzburg
Copy link
Author

oliversalzburg commented May 28, 2021

@isidorn I didn't expect the hotkey to work when I have the editor in focus. I expected it to work when the file explorer panel is in focus. That expectation was and is not met.

For clarification: I click on a file in the file explorer panel. Then I press the key combination. Nothing happens.

I can click the file as many times as I want (in case the focus was moved elsewhere) and it does not work.

If my approach is not right, please let me know the right way.

Also, I'm aware of the ability to reconfigure this behavior, but I felt like this is an issue worth reporting. Especially since focus and hot key related issues seem to be kind of my thing.

@isidorn
Copy link
Contributor

isidorn commented May 28, 2021

@oliversalzburg ok. Can you reproduce with VS Code insiders?
Since this just works for me

@oliversalzburg
Copy link
Author

@isidorn For me the behavior is identical in Insiders. I should have noted that earlier.

Version: 1.57.0-insider (user setup)
Commit: 541a5d4
Date: 2021-05-28T09:50:57.063Z
Electron: 12.0.9
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: 8.9.255.25-electron.0
OS: Windows_NT x64 10.0.19042

@oliversalzburg
Copy link
Author

oliversalzburg commented May 28, 2021

I just noticed this works as intended on Windows. I'm experiencing the issue when working in WSL.

The same project will behave differently depending on which environment I open it in.

And, just for clarification, the context menu entry does open the selected file/folder reliably in both environments.

@isidorn
Copy link
Contributor

isidorn commented May 28, 2021

Ok, so the gist of the issue seems to be "Reveal in Explorer does not work for WSL". Let me reopen...

@isidorn isidorn reopened this May 28, 2021
@isidorn isidorn added under-discussion Issue is under discussion for relevance, priority, approach and removed *as-designed Described behavior is as designed labels May 28, 2021
@isidorn isidorn added the *out-of-scope Posted issue is not in scope of VS Code label Aug 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
file-explorer Explorer widget issues *out-of-scope Posted issue is not in scope of VS Code under-discussion Issue is under discussion for relevance, priority, approach
Projects
None yet
Development

No branches or pull requests

3 participants
@oliversalzburg @isidorn and others