Fix handling of default factory in the URL (#6873)

jupyter · May 16, 2023 · e976dfd · e976dfd
1 parent d518364
commit e976dfd
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/packages/docmanager-extension/src/index.ts b/packages/docmanager-extension/src/index.ts
@@ -24,6 +24,7 @@ const opener: JupyterFrontEndPlugin<IDocumentWidgetOpener> = {
   provides: IDocumentWidgetOpener,
   activate: (app: JupyterFrontEnd) => {
     const baseUrl = PageConfig.getBaseUrl();
+    const docRegistry = app.docRegistry;
     let id = 0;
     return new (class {
       open(widget: IDocumentWidget, options?: DocumentRegistry.IOpenOptions) {
@@ -42,7 +43,8 @@ const opener: JupyterFrontEndPlugin<IDocumentWidgetOpener> = {
           }
           let url = `${baseUrl}${route}/${path}`;
           // append ?factory only if it's not the default
-          if (widgetName !== 'default') {
+          const defaultFactory = docRegistry.defaultWidgetFactory(path);
+          if (widgetName !== defaultFactory.name) {
             url = `${url}?factory=${widgetName}`;
           }
           window.open(url);