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

toggle problems shortcut #71790

Closed
jerrygreen opened this issue Apr 5, 2019 · 10 comments
Closed

toggle problems shortcut #71790

jerrygreen opened this issue Apr 5, 2019 · 10 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug error-list Problems view verified Verification succeeded
Milestone

Comments

@jerrygreen
Copy link

jerrygreen commented Apr 5, 2019

It's not toggling back (not closing the "problems" view)

Versions

  • VSCode Version: 1.32.3
  • OS Version: macOS Mojave 10.14.3

Steps to Reproduce:

  1. cmd + shift + m to toggle the view (it works)
  2. cmd + shift + m to toggle it back / to close (it won't work)

Does this issue occur when all extensions are disabled?

Yes

@vscodebot vscodebot bot added the extensions Issues concerning extensions label Apr 5, 2019
@jerrygreen
Copy link
Author

jerrygreen commented Apr 5, 2019

However, if I use a separate tab, "output" (cmd + shift + u) – it toggles back and forth

But cmd + shift + m is one-side toggle :(

@jwikman
Copy link
Contributor

jwikman commented Apr 7, 2019

This has annoyed me as well.

Actually I think it's related to the behavior of the Problems panel. If there are no problems, the "problems" panel isn't focused on ctrl+shift+m. As soon as I got a problem this works as intended. You can see the focusing of the panel (or lack of it) by observing the border of that panel.

The panel is only hidden if it is focused as seen in src\vs\workbench\browser\panel.ts:

		if (this.isPanelFocused()) {
			this.layoutService.setPanelHidden(true);
		} else {
			this.panelService.openPanel(this.panelId, true);
		}

So I think the the solution would be to focus on the "problems" panel even if there are no problems.

@sandy081
Copy link
Member

sandy081 commented Apr 8, 2019

@jerrygreen Can you please check in your keybindings editor to which command ctrl+shift+m keybinding is associated to?

@sandy081 sandy081 added info-needed Issue requires more information from poster and removed extensions Issues concerning extensions labels Apr 8, 2019
@jwikman
Copy link
Contributor

jwikman commented Apr 8, 2019

@sandy081 I've got the default binding, with a similar experience as @jerrygreen
But I only experience this if the problems list is empty, don't know if @jerrygreen has this issue also when there are problems...

Keybinding:

{
  "key": "ctrl+shift+m",
  "command": "workbench.actions.view.problems"
}

@sandy081
Copy link
Member

sandy081 commented Apr 8, 2019

@jwikman I see following as default

image

Can you disable all extensions and check again?

@jerrygreen
Copy link
Author

@sandy081 ofc:

image

Also, agree with @jwikman:

But I only experience this if the problems list is empty

@jwikman
Copy link
Contributor

jwikman commented Apr 8, 2019

@jwikman I see following as default

image

Can you disable all extensions and check again?

@sandy081 if you right click and select copy you get the same result as me... I.e. I haven't changed any keybinding, I'm using the default ones. If you search for "workbench.actions.view.problems" in the code, you'll see that's the default command for that ctrl+shift+M.

So no surprises here: it's the same with all extensions disabled...

@sandy081
Copy link
Member

sandy081 commented Apr 9, 2019

Ok, will investigate why it is behaving differently when problems are empty.

@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug error-list Problems view labels Apr 9, 2019
@sandy081 sandy081 added this to the On Deck milestone Apr 9, 2019
@sandy081 sandy081 removed the info-needed Issue requires more information from poster label Apr 9, 2019
@chrisjones-brack3t
Copy link

Also running into this issue and it's driving me a little mad.

@akshaal
Copy link

akshaal commented May 24, 2019

Quite annoying. Is there a workaround at the moment? It could have been possible to just call workbench.action.togglePanel when the problems panel is already visible, but it looks like there is no way to find out whether the problems panel is visible or not....

@sandy081 sandy081 modified the milestones: On Deck, May 2019 May 28, 2019
@Tyriar Tyriar added the verified Verification succeeded label May 29, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Jul 12, 2019
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 error-list Problems view verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

7 participants
@chrisjones-brack3t @akshaal @Tyriar @sandy081 @jerrygreen @jwikman and others