Disable some commands while in the walkthrough

src/vs/editor/common/editorCommon.ts

@@ -3129,6 +3129,10 @@ export namespace ModeContextKeys {
 	 * @internal
 	 */
 	export const hasSignatureHelpProvider = new RawContextKey<boolean>('editorHasSignatureHelpProvider', undefined);
+	/**
+	 * @internal
+	 */
+	export const isInWalkThrough = new RawContextKey<boolean>('isInWalkThrough', undefined);
 }
 
 

src/vs/editor/common/modes/editorModeContext.ts

@@ -28,6 +28,7 @@ export class EditorModeContext {
 	private _hasDocumentFormattingProvider: IContextKey<boolean>;
 	private _hasDocumentSelectionFormattingProvider: IContextKey<boolean>;
 	private _hasSignatureHelpProvider: IContextKey<boolean>;
+	private _isInWalkThrough: IContextKey<boolean>;
 
 	constructor(
 		editor: ICommonCodeEditor,
@@ -49,6 +50,7 @@ export class EditorModeContext {
 		this._hasDocumentFormattingProvider = ModeContextKeys.hasDocumentFormattingProvider.bindTo(contextKeyService);
 		this._hasDocumentSelectionFormattingProvider = ModeContextKeys.hasDocumentSelectionFormattingProvider.bindTo(contextKeyService);
 		this._hasSignatureHelpProvider = ModeContextKeys.hasSignatureHelpProvider.bindTo(contextKeyService);
+		this._isInWalkThrough = ModeContextKeys.isInWalkThrough.bindTo(contextKeyService);
 
 		// update when model/mode changes
 		this._disposables.push(editor.onDidChangeModel(() => this._update()));
@@ -91,6 +93,7 @@ export class EditorModeContext {
 		this._hasDocumentFormattingProvider.reset();
 		this._hasDocumentSelectionFormattingProvider.reset();
 		this._hasSignatureHelpProvider.reset();
+		this._isInWalkThrough.reset();
 	}
 
 	private _update() {
@@ -113,5 +116,6 @@ export class EditorModeContext {
 		this._hasSignatureHelpProvider.set(modes.SignatureHelpProviderRegistry.has(model));
 		this._hasDocumentFormattingProvider.set(modes.DocumentFormattingEditProviderRegistry.has(model) || modes.DocumentRangeFormattingEditProviderRegistry.has(model));
 		this._hasDocumentSelectionFormattingProvider.set(modes.DocumentRangeFormattingEditProviderRegistry.has(model));
+		this._isInWalkThrough.set(model.uri.scheme === 'walkThroughSnippet');
 	}
 }

src/vs/editor/contrib/goToDeclaration/browser/goToDeclaration.ts

@@ -169,7 +169,9 @@ export class GoToDefinitionAction extends DefinitionAction {
 			id: GoToDefinitionAction.ID,
 			label: nls.localize('actions.goToDecl.label', "Go to Definition"),
 			alias: 'Go to Definition',
-			precondition: ModeContextKeys.hasDefinitionProvider,
+			precondition: ContextKeyExpr.and(
+				ModeContextKeys.hasDefinitionProvider,
+				ModeContextKeys.isInWalkThrough.toNegated()),
 			kbOpts: {
 				kbExpr: EditorContextKeys.TextFocus,
 				primary: goToDeclarationKb
@@ -192,7 +194,9 @@ export class OpenDefinitionToSideAction extends DefinitionAction {
 			id: OpenDefinitionToSideAction.ID,
 			label: nls.localize('actions.goToDeclToSide.label', "Open Definition to the Side"),
 			alias: 'Open Definition to the Side',
-			precondition: ModeContextKeys.hasDefinitionProvider,
+			precondition: ContextKeyExpr.and(
+				ModeContextKeys.hasDefinitionProvider,
+				ModeContextKeys.isInWalkThrough.toNegated()),
 			kbOpts: {
 				kbExpr: EditorContextKeys.TextFocus,
 				primary: KeyChord(KeyMod.CtrlCmd | KeyCode.KEY_K, goToDeclarationKb)
@@ -208,7 +212,10 @@ export class PeekDefinitionAction extends DefinitionAction {
 			id: 'editor.action.previewDeclaration',
 			label: nls.localize('actions.previewDecl.label', "Peek Definition"),
 			alias: 'Peek Definition',
-			precondition: ContextKeyExpr.and(ModeContextKeys.hasDefinitionProvider, PeekContext.notInPeekEditor),
+			precondition: ContextKeyExpr.and(
+				ModeContextKeys.hasDefinitionProvider,
+				PeekContext.notInPeekEditor,
+				ModeContextKeys.isInWalkThrough.toNegated()),
 			kbOpts: {
 				kbExpr: EditorContextKeys.TextFocus,
 				primary: KeyMod.Alt | KeyCode.F12,
@@ -233,7 +240,9 @@ export class GoToImplementationAction extends DefinitionAction {
 			id: GoToImplementationAction.ID,
 			label: nls.localize('actions.goToImplementation.label', "Go to Implementation"),
 			alias: 'Go to Implementation',
-			precondition: ModeContextKeys.hasTypeDefinitionProvider,
+			precondition: ContextKeyExpr.and(
+				ModeContextKeys.hasTypeDefinitionProvider,
+				ModeContextKeys.isInWalkThrough.toNegated()),
 			kbOpts: {
 				kbExpr: EditorContextKeys.TextFocus,
 				primary: KeyMod.CtrlCmd | KeyCode.F12
@@ -260,7 +269,9 @@ export class PeekImplementationAction extends DefinitionAction {
 			id: PeekImplementationAction.ID,
 			label: nls.localize('actions.peekImplementation.label', "Peek Implementation"),
 			alias: 'Peek Implementation',
-			precondition: ModeContextKeys.hasTypeDefinitionProvider,
+			precondition: ContextKeyExpr.and(
+				ModeContextKeys.hasTypeDefinitionProvider,
+				ModeContextKeys.isInWalkThrough.toNegated()),
 			kbOpts: {
 				kbExpr: EditorContextKeys.TextFocus,
 				primary: KeyMod.CtrlCmd | KeyMod.Shift | KeyCode.F12

src/vs/editor/contrib/referenceSearch/browser/referenceSearch.ts

@@ -63,7 +63,10 @@ export class ReferenceAction extends EditorAction {
 			id: 'editor.action.referenceSearch.trigger',
 			label: nls.localize('references.action.label', "Find All References"),
 			alias: 'Find All References',
-			precondition: ContextKeyExpr.and(ModeContextKeys.hasReferenceProvider, PeekContext.notInPeekEditor),
+			precondition: ContextKeyExpr.and(
+				ModeContextKeys.hasReferenceProvider,
+				PeekContext.notInPeekEditor,
+				ModeContextKeys.isInWalkThrough.toNegated()),
 			kbOpts: {
 				kbExpr: EditorContextKeys.TextFocus,
 				primary: KeyMod.Shift | KeyCode.F12