Feb 19, 2024 · Feb 16, 2024 · Feb 16, 2024 · Feb 18, 2024 · Feb 18, 2024 · Feb 18, 2024
diff --git a/packages/notebook-extension/src/index.ts b/packages/notebook-extension/src/index.ts
    translator: ITranslator | null,
    notebookTools: INotebookTools | null
  ) => {
    const { commands } = app;
    const { commands, shell } = app;
    translator = translator ?? nullTranslator;
    const trans = translator.load('notebook');

          });
        }
      },
      isEnabled: () => { app.shell.currentWidget !== null &&
          app.shell.currentWidget instanceof NotebookPanel }
      isEnabled: () => shell.currentWidget !== null && shell.currentWidget instanceof NotebookPanel
    });

    if (palette) {
Original file line number	Diff line number	Diff line change
Expand Up		@@ -517,7 +517,7 @@ const editNotebookMetadata: JupyterFrontEndPlugin<void> = {
		translator: ITranslator \| null,
		notebookTools: INotebookTools \| null
		) => {
		const { commands } = app;
		const { commands, shell } = app;
		translator = translator ?? nullTranslator;
		const trans = translator.load('notebook');

Expand DownExpand Up		@@ -549,8 +549,7 @@ const editNotebookMetadata: JupyterFrontEndPlugin<void> = {
		});
		}
		},
		isEnabled: () => { app.shell.currentWidget !== null &&
		app.shell.currentWidget instanceof NotebookPanel }
		isEnabled: () => shell.currentWidget !== null && shell.currentWidget instanceof NotebookPanel
Copy link Member krassowskiFeb 18, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. This will need adding some new lines to satisfy linter. I would consider going as far as hiding it altogether (`isVisible`?). itsmevichu reacted with thumbs up emoji Copy link ContributorAuthor itsmevichuFeb 18, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Sure, I will incorporate the changes.
		});

		if (palette) {
Expand Down