Disable run/interrupt/restart buttons when switching kernels

news/2 Fixes/9935.md

@@ -0,0 +1 @@
+Switching kernels should disable the run/interrupt/restart buttons.

src/client/datascience/interactive-common/interactiveBase.ts

@@ -1480,7 +1480,12 @@ export abstract class InteractiveBase extends WebViewHost<IInteractiveWindowMapp
         if (!this._notebook) {
             return;
         }
-        await this.commandManager.executeCommand(Commands.SwitchJupyterKernel, this._notebook);
+        try {
+            this.startProgress();
+            await this.commandManager.executeCommand(Commands.SwitchJupyterKernel, this._notebook);
+        } finally {
+            this.stopProgress();
+        }
     }
     private async kernelChangeHandler(kernel: IJupyterKernelSpec | LiveKernelModel) {
         // Check if we are changing to LiveKernelModel

src/datascience-ui/native-editor/nativeCell.tsx

@@ -43,6 +43,7 @@ interface INativeCellBaseProps {
     editorOptions: monacoEditor.editor.IEditorOptions;
     themeMatplotlibPlots: boolean | undefined;
     focusPending: number;
+    busy: boolean;
 }
 
 type INativeCellProps = INativeCellBaseProps & typeof actionCreators;
@@ -567,6 +568,7 @@ export class NativeCell extends React.Component<INativeCellProps> {
                         onClick={runCell}
                         tooltip={getLocString('DataScience.runCell', 'Run cell')}
                         hidden={this.isMarkdownCell()}
+                        disabled={this.props.busy}
                     >
                         <Image baseTheme={this.props.baseTheme} class="image-button-image" image={ImageName.Run} />
                     </ImageButton>

src/datascience-ui/native-editor/nativeEditor.tsx

@@ -477,6 +477,7 @@ ${buildSettingsCss(this.props.settings)}`}</style>
                         themeMatplotlibPlots={this.props.settings.themeMatplotlibPlots}
                         // Focus pending does not apply to native editor.
                         focusPending={0}
+                        busy={this.props.busy}
                     />
                 </ErrorBoundary>
                 {lastLine}