Always consider the "passthrough" commmand enabled for keybindings (#13564)

tsmaeder · jfaltermeier · commit 49e167c6d7de · 2024-04-05T09:42:43.000+02:00
Fixes #13560 Contributed on behalf of STMicroelectronics Signed-off-by: Thomas Mäder <t.s.maeder@gmail.com>
diff --git a/packages/core/src/browser/keybinding.ts b/packages/core/src/browser/keybinding.ts
@@ -497,7 +497,7 @@ export class KeybindingRegistry {
 
     isEnabledInScope(binding: common.Keybinding, target: HTMLElement | undefined): boolean {
         const context = binding.context && this.contexts[binding.context];
-        if (binding.command && !this.commandRegistry.isEnabled(binding.command, binding.args)) {
+        if (binding.command && (!this.isPseudoCommand(binding.command) && !this.commandRegistry.isEnabled(binding.command, binding.args))) {
             return false;
         }
         if (context && !context.isEnabled(binding)) {

Original file line number	Diff line number	Diff line change
`@@ -497,7 +497,7 @@ export class KeybindingRegistry {`
`497`	`497`
`498`	`498`	`isEnabledInScope(binding: common.Keybinding, target: HTMLElement \| undefined): boolean {`
`499`	`499`	`const context = binding.context && this.contexts[binding.context];`
`500`		`- if (binding.command && !this.commandRegistry.isEnabled(binding.command, binding.args)) {`
	`500`	`+ if (binding.command && (!this.isPseudoCommand(binding.command) && !this.commandRegistry.isEnabled(binding.command, binding.args))) {`
`501`	`501`	`return false;`
`502`	`502`	`}`
`503`	`503`	`if (context && !context.isEnabled(binding)) {`