-
Notifications
You must be signed in to change notification settings - Fork 28.7k
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
The sidebar is throwing errors like crazy in VSCode 1.6.0 #13447
Comments
btw... these errors happen even with the |
Does |
Just ran into this myself with 1.6. With --force-gpu-rasterization I get:
|
@chrmarti - I tried |
Possibly related to #12709 |
I at least do not have the REST Client extension installed, and |
@TimMensch Which OS and OS version are you running? |
Windows 10/x64, version 1.6.0, 2016-10-10T18:37:40 |
Sorry, OS version: Windows 10 Pro, 1511, 10586.589 |
@chrmarti I've got another Windows 10 system, but I've not let VS Code update on it for fear of losing search on both. |
I uninstalled all of my extensions, just to see if that helped, but no luck. |
Could you append your keybindings.json? There appears to be a bug with the new commands in the Search viewlet. @sandy081 |
I'm not modifying the "Find" key:
|
@chrmarti For fun I deleted my keybindings and it still comes up with the same error, and selecting |
@chrmarti - Yep. I was just about to post that. I completely uninstalled VSCode and deleted my Then I restored my custom keybindings and restarted VSCode, and the errors were back. 👎 So I started removing custom keybindings one-by-one and restarting VSCode after each one until I found the culprit. Here is the keybinding that's causing the error (for me, anyway)... // Place your key bindings in this file to overwrite the defaults
[
{
"key": "alt+cmd+w",
"command": "workbench.action.closeAllEditors"
}
] |
@BigstickCarpet @chrmarti Sure enough, success:
This was the culprit for me. It didn't fix things to just delete it; I had to restart VS Code between tests. |
A workaround would be to make sure the following commands are bound to at least one (arbitrary) key combination:
And, as you mentioned, a restart is necessary to make it work. |
Awesome sleuthing. 🕵️ This also works for me:
I added a "when" clause to my keybinding so it wouldn't kill the other binding, just hide it when the editor is active. |
/cc @isidorn |
Problem for me was also a custom keybinding: Presumably because it overwrites the default for toggleFindRegex and thus invalidates the requirements specified by @chrmarti : |
To me it only happens when assigns to left menu empty {
"key": "cmd+alt+r",
"command": "editor.action.format"
} left menu ok {
"key": "cmd+r",
"command": "editor.action.format"
} |
@eduardonunesp On Mac, cmd+alt+r is the short cut for enabling reg-ex Thanks |
Problem for me was due to:
|
Same here on MacOS El Capitan my keybindings.json was
After delete the custom keybindings and restart, everything works fine. |
@d4v1dst Sorry for that. This is because of the key-binding |
Verification: Customize following default key bindings to other actions Mac
Windows/Linux
Verify that Search view is opening without any errors |
For me, these custom keybindings were the issue, after removing them, the search view worked: { "key": "shift+alt+c", "command": "toggleFindCaseSensitive", "when": "editorFocus" }, Also, I need to mention that I have another custom key bound to alt+c, that before the update of vs code to 1.6.0, was a free key: |
@lypscan alt+c was always taken to toggle case sensitive option in Editor find widget. With your customization, this key binding was overridden |
@sandy081 yes, now I remember... In any case, the error seems to manifest itself whenever I override the default keybindings for: toggleFindCaseSensitive, toggleFindWholeWord, toggleFindRegex. |
This behavior started happening after upgrading to VSCode 1.6.0 today.
Steps to Reproduce:
1. Search pane is blank
2. Cannot read property 'focus' of undefined
cmd+shift+f
to trigger theworkbench.view.search
command3. Cannot read property 'toCustomLabel' of undefined
cmd+shift+f
to trigger theworkbench.view.search
command4. Cannot read property 'setWidth' of undefined
cmd+b
to trigger theworkbench.action.toggleSidebarVisibility
commandThe text was updated successfully, but these errors were encountered: