Skip to content

Commit

Permalink
Phosphore widget backward compatibility fix on WidgetRenderer
Browse files Browse the repository at this point in the history
  • Loading branch information
martinRenou committed Aug 19, 2024
1 parent c579fcd commit b76c1c2
Show file tree
Hide file tree
Showing 4 changed files with 42 additions and 36 deletions.
4 changes: 2 additions & 2 deletions docs/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ ipyleaflet
jupyter-client
jupyter-packaging
jupyterlab >=4
jupyterlite-core >=0.3.0<0.4.0
jupyterlite-pyodide-kernel >=0.3.0<0.4.0
jupyterlite-core >=0.3.0,<0.4.0
jupyterlite-pyodide-kernel >=0.3.0,<0.4.0
matplotlib
myst-nb >=0.17,<0.18
numpy
Expand Down
5 changes: 5 additions & 0 deletions packages/base/src/widget.ts
Original file line number Diff line number Diff line change
Expand Up @@ -983,6 +983,11 @@ export class JupyterLuminoPanelWidget extends Panel {
private _view: DOMWidgetView;
}

/**
* @deprecated Use {@link JupyterLuminoPanelWidget} instead (Since 8.0).
*/
export const JupyterPhosphorPanelWidget = JupyterLuminoPanelWidget;

export class DOMWidgetView extends WidgetView {
/**
* Public constructor
Expand Down
3 changes: 2 additions & 1 deletion python/jupyterlab_widgets/src/renderer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,8 @@ export class WidgetRenderer

let widget: LuminoWidget;
try {
widget = (await manager.create_view(wModel)).luminoWidget;
const view = await manager.create_view(wModel);
widget = view.luminoWidget || view.pWidget;
} catch (err) {
this.node.textContent = 'Error displaying widget';
this.addClass('jupyter-widgets');
Expand Down
66 changes: 33 additions & 33 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -732,11 +732,11 @@ __metadata:
languageName: node
linkType: hard

"@jupyter-widgets/base-manager@^1.0.8, @jupyter-widgets/base-manager@workspace:packages/base-manager":
"@jupyter-widgets/base-manager@^1.0.9, @jupyter-widgets/base-manager@workspace:packages/base-manager":
version: 0.0.0-use.local
resolution: "@jupyter-widgets/base-manager@workspace:packages/base-manager"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base": ^6.0.8
"@jupyterlab/services": ^6.0.0 || ^7.0.0
"@lumino/coreutils": ^1.11.1 || ^2
"@types/base64-js": ^1.2.5
Expand Down Expand Up @@ -771,7 +771,7 @@ __metadata:
languageName: unknown
linkType: soft

"@jupyter-widgets/base@^6.0.7, @jupyter-widgets/base@workspace:packages/base":
"@jupyter-widgets/base@^6.0.8, @jupyter-widgets/base@workspace:packages/base":
version: 0.0.0-use.local
resolution: "@jupyter-widgets/base@workspace:packages/base"
dependencies:
Expand Down Expand Up @@ -814,11 +814,11 @@ __metadata:
languageName: unknown
linkType: soft

"@jupyter-widgets/controls@^5.0.8, @jupyter-widgets/controls@workspace:packages/controls":
"@jupyter-widgets/controls@^5.0.9, @jupyter-widgets/controls@workspace:packages/controls":
version: 0.0.0-use.local
resolution: "@jupyter-widgets/controls@workspace:packages/controls"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base": ^6.0.8
"@jupyterlab/services": ^6.0.0 || ^7.0.0
"@lumino/algorithm": ^1.9.1 || ^2.1
"@lumino/domutils": ^1.8.1 || ^2.1
Expand Down Expand Up @@ -869,9 +869,9 @@ __metadata:
version: 0.0.0-use.local
resolution: "@jupyter-widgets/example-web1@workspace:examples/web1"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base-manager": ^1.0.8
"@jupyter-widgets/controls": ^5.0.8
"@jupyter-widgets/base": ^6.0.8
"@jupyter-widgets/base-manager": ^1.0.9
"@jupyter-widgets/controls": ^5.0.9
chai: ^4.0.0
css-loader: ^6.5.1
karma: ^6.3.3
Expand All @@ -890,9 +890,9 @@ __metadata:
version: 0.0.0-use.local
resolution: "@jupyter-widgets/example-web2@workspace:examples/web2"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base-manager": ^1.0.8
"@jupyter-widgets/controls": ^5.0.8
"@jupyter-widgets/base": ^6.0.8
"@jupyter-widgets/base-manager": ^1.0.9
"@jupyter-widgets/controls": ^5.0.9
codemirror: ^5.48.0
css-loader: ^6.5.1
font-awesome: ^4.7.0
Expand All @@ -905,9 +905,9 @@ __metadata:
version: 0.0.0-use.local
resolution: "@jupyter-widgets/example-web3@workspace:examples/web3"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/controls": ^5.0.8
"@jupyter-widgets/html-manager": ^1.0.10
"@jupyter-widgets/base": ^6.0.8
"@jupyter-widgets/controls": ^5.0.9
"@jupyter-widgets/html-manager": ^1.0.11
"@jupyterlab/services": ^6.0.0 || ^7.0.0
"@types/codemirror": ^5.60.0
"@types/node": ^17.0.2
Expand All @@ -927,24 +927,24 @@ __metadata:
version: 0.0.0-use.local
resolution: "@jupyter-widgets/example-web4@workspace:examples/web4"
dependencies:
"@jupyter-widgets/html-manager": ^1.0.10
"@jupyter-widgets/html-manager": ^1.0.11
css-loader: ^6.5.1
font-awesome: ^4.7.0
style-loader: ^3.3.1
webpack: ^5.65.0
languageName: unknown
linkType: soft

"@jupyter-widgets/html-manager@^1.0.10, @jupyter-widgets/html-manager@workspace:packages/html-manager":
"@jupyter-widgets/html-manager@^1.0.11, @jupyter-widgets/html-manager@workspace:packages/html-manager":
version: 0.0.0-use.local
resolution: "@jupyter-widgets/html-manager@workspace:packages/html-manager"
dependencies:
"@fortawesome/fontawesome-free": ^5.12.0
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base-manager": ^1.0.8
"@jupyter-widgets/controls": ^5.0.8
"@jupyter-widgets/output": ^6.0.7
"@jupyter-widgets/schema": ^0.5.4
"@jupyter-widgets/base": ^6.0.8
"@jupyter-widgets/base-manager": ^1.0.9
"@jupyter-widgets/controls": ^5.0.9
"@jupyter-widgets/output": ^6.0.8
"@jupyter-widgets/schema": ^0.5.5
"@jupyterlab/outputarea": ^3.0.0 || ^4.0.0
"@jupyterlab/rendermime": ^3.0.0 || ^4.0.0
"@jupyterlab/rendermime-interfaces": ^3.0.0 || ^4.0.0
Expand Down Expand Up @@ -981,10 +981,10 @@ __metadata:
version: 0.0.0-use.local
resolution: "@jupyter-widgets/jupyterlab-manager@workspace:python/jupyterlab_widgets"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base-manager": ^1.0.8
"@jupyter-widgets/controls": ^5.0.8
"@jupyter-widgets/output": ^6.0.7
"@jupyter-widgets/base": ^6.0.8
"@jupyter-widgets/base-manager": ^1.0.9
"@jupyter-widgets/controls": ^5.0.9
"@jupyter-widgets/output": ^6.0.8
"@jupyterlab/application": ^3.0.0 || ^4.0.0
"@jupyterlab/apputils": ^3.0.0 || ^4.0.0
"@jupyterlab/builder": ^3.0.0 || ^4.0.0
Expand Down Expand Up @@ -1028,11 +1028,11 @@ __metadata:
version: 0.0.0-use.local
resolution: "@jupyter-widgets/notebook-manager@workspace:python/widgetsnbextension"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base-manager": ^1.0.8
"@jupyter-widgets/controls": ^5.0.8
"@jupyter-widgets/html-manager": ^1.0.10
"@jupyter-widgets/output": ^6.0.7
"@jupyter-widgets/base": ^6.0.8
"@jupyter-widgets/base-manager": ^1.0.9
"@jupyter-widgets/controls": ^5.0.9
"@jupyter-widgets/html-manager": ^1.0.11
"@jupyter-widgets/output": ^6.0.8
"@jupyterlab/services": ^6.0.0 || ^7.0.0
"@lumino/messaging": ^1.10.1 || ^2.1
"@lumino/widgets": ^1.30.0 || ^2.1
Expand All @@ -1046,17 +1046,17 @@ __metadata:
languageName: unknown
linkType: soft

"@jupyter-widgets/output@^6.0.7, @jupyter-widgets/output@workspace:packages/output":
"@jupyter-widgets/output@^6.0.8, @jupyter-widgets/output@workspace:packages/output":
version: 0.0.0-use.local
resolution: "@jupyter-widgets/output@workspace:packages/output"
dependencies:
"@jupyter-widgets/base": ^6.0.7
"@jupyter-widgets/base": ^6.0.8
rimraf: ^3.0.2
typescript: ~4.9.4
languageName: unknown
linkType: soft

"@jupyter-widgets/schema@^0.5.4, @jupyter-widgets/schema@workspace:packages/schema":
"@jupyter-widgets/schema@^0.5.5, @jupyter-widgets/schema@workspace:packages/schema":
version: 0.0.0-use.local
resolution: "@jupyter-widgets/schema@workspace:packages/schema"
languageName: unknown
Expand Down

0 comments on commit b76c1c2

Please sign in to comment.