Browse Source

Explicitly create a Panel instance for each ConsolePanel instead of passing it in.

Afshin T. Darian 5 years ago
parent
commit
7e4e796f15

+ 1 - 2
examples/console/src/index.ts

@@ -12,7 +12,7 @@ import '../index.css';
 
 import { CommandRegistry } from '@lumino/commands';
 
-import { CommandPalette, SplitPanel, Widget, Panel } from '@lumino/widgets';
+import { CommandPalette, SplitPanel, Widget } from '@lumino/widgets';
 
 import { ServiceManager } from '@jupyterlab/services';
 
@@ -70,7 +70,6 @@ function startApp(path: string, manager: ServiceManager.IManager) {
   let editorFactory = editorServices.factoryService.newInlineEditor;
   let contentFactory = new ConsolePanel.ContentFactory({ editorFactory });
   let consolePanel = new ConsolePanel({
-    content: new Panel(),
     rendermime,
     manager,
     path,

+ 0 - 1
packages/completer-extension/src/index.ts

@@ -140,7 +140,6 @@ const consoles: JupyterFrontEndPlugin<void> = {
       const editor = anchor.promptCell?.editor ?? null;
       const session = anchor.sessionContext.session;
       // TODO: CompletionConnector assumes editor and session are not null
-
       const connector = new CompletionConnector({ session, editor });
       const handler = manager.register({ connector, editor, parent: widget });
 

+ 1 - 2
packages/console-extension/src/index.ts

@@ -53,7 +53,7 @@ import {
 
 import { DisposableSet } from '@lumino/disposable';
 
-import { DockLayout, Menu, Panel } from '@lumino/widgets';
+import { DockLayout, Menu } from '@lumino/widgets';
 
 import foreign from './foreign';
 
@@ -249,7 +249,6 @@ async function activateConsole(
     await manager.ready;
 
     const panel = new ConsolePanel({
-      content: new Panel(),
       manager,
       contentFactory,
       mimeTypeService: editorServices.mimeTypeService,

+ 1 - 2
packages/console/src/panel.ts

@@ -45,7 +45,7 @@ export class ConsolePanel extends MainAreaWidget<Panel> {
    * Construct a console panel.
    */
   constructor(options: ConsolePanel.IOptions) {
-    super(options);
+    super({ content: new Panel() });
     this.addClass(PANEL_CLASS);
     let {
       rendermime,
@@ -178,7 +178,6 @@ export namespace ConsolePanel {
    * The initialization options for a console panel.
    */
   export interface IOptions {
-    content: Panel;
     /**
      * The rendermime instance used by the panel.
      */

+ 1 - 2
tests/test-console/src/panel.spec.ts

@@ -7,7 +7,7 @@ import { ServiceManager } from '@jupyterlab/services';
 
 import { Message, MessageLoop } from '@lumino/messaging';
 
-import { Widget, Panel } from '@lumino/widgets';
+import { Widget } from '@lumino/widgets';
 
 import { CodeConsole, ConsolePanel } from '@jupyterlab/console';
 
@@ -46,7 +46,6 @@ describe('console/panel', () => {
 
   beforeEach(() => {
     panel = new TestPanel({
-      content: new Panel(),
       manager,
       contentFactory,
       rendermime,