Skip to content

JupyterLab 3.0 #112

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Oct 22, 2021
Merged

JupyterLab 3.0 #112

merged 7 commits into from
Oct 22, 2021

Conversation

khwj
Copy link

@khwj khwj commented Feb 26, 2021

@khwj khwj marked this pull request as ready for review February 26, 2021 16:23
@beenje
Copy link

beenje commented Mar 1, 2021

The errorPanel isn't working: Unhandled Promise Rejection: NotFoundError: The object can not be found here. on RemoveChild.

If you know how to fix it, I'm interested :-)
I have the same issue for the jupyterlab-gitlab extension.

@jingw222
Copy link

Has the issue been suspended? Still not fixed as of yet.

Extension Installation Error
An error occurred installing <code>@jupyterlab/github</code>.

Error message:

The extension "@jupyterlab/github" does not yet support the current version of JupyterLab.


Conflicting Dependencies:
JupyterLab                  Extension       Package
>=3.0.10 <3.1.0             >=2.0.0 <3.0.0  @jupyterlab/application
>=3.0.8 <3.1.0              >=2.0.0 <3.0.0  @jupyterlab/apputils
>=5.0.6 <5.1.0              >=4.0.0 <5.0.0  @jupyterlab/coreutils
>=3.0.10 <3.1.0             >=2.0.0 <3.0.0  @jupyterlab/docmanager
>=3.0.10 <3.1.0             >=2.0.0 <3.0.0  @jupyterlab/filebrowser
>=6.0.8 <6.1.0              >=5.0.0 <6.0.0  @jupyterlab/services
>=3.0.5 <3.1.0              >=2.0.0 <3.0.0  @jupyterlab/settingregistry
>=3.0.7 <3.1.0              >=2.0.0 <3.0.0  @jupyterlab/ui-components

@jhgoebbert
Copy link

jhgoebbert commented Aug 17, 2021

The errorPanel isn't working: Unhandled Promise Rejection: NotFoundError: The object can not be found here. on RemoveChild.

If you know how to fix it, I'm interested :-)
I have the same issue for the jupyterlab-gitlab extension.

The following line
const listing = (this._browser.layout as PanelLayout).widgets[2];
can be found at these four places.
https://github.com/jupyterlab/jupyterlab-github/blob/v2.0.0/src/browser.ts#L149
https://github.com/jupyterlab/jupyterlab-github/blob/v2.0.0/src/browser.ts#L228
https://github.com/jupyterlab/jupyterlab-github/blob/v2.0.0/src/browser.ts#L241
https://github.com/jupyterlab/jupyterlab-github/blob/v2.0.0/src/browser.ts#L252

It assumes, that the 3rd widget (=[2]) of PanelLayout ist listing.
This was the case until JupyterLab 2.x for its FileBrowser:
https://github.com/jupyterlab/jupyterlab/blob/2.3.x/packages/filebrowser/src/browser.ts#L101

But now the widget listing is the 4th one (=[3]), because _filenameSearcher is new.
https://github.com/jupyterlab/jupyterlab/blob/3.1.x/packages/filebrowser/src/browser.ts#L129

Hence, I expect it now should be
const listing = (this._browser.layout as PanelLayout).widgets[3];

@jhgoebbert
Copy link

I checked, if [2] -> [3] fixes the errorPanel-bug and it does. 👍

@krassowski
Copy link
Member

listing is no longer private (only protected), but since this extension only uses the HTML node to focus on it, it could just:

const LISTING_CLASS = 'jp-FileBrowser-listing';
this._browser.node.querySelector('.' + LISTING_CLASS).focus()

@shadow150519
Copy link

has this problem been fixed yet ? i also encounter the same problem

Extension Installation Error
An error occurred installing <code>@jupyterlab/github</code>.

Error message:

The extension "@jupyterlab/github" does not yet support the current version of JupyterLab.


Conflicting Dependencies:
JupyterLab                  Extension      Package
>=3.1.3 <3.2.0              >=2.0.0 <3.0.0 @jupyterlab/application
>=3.1.3 <3.2.0              >=2.0.0 <3.0.0 @jupyterlab/apputils
>=5.1.3 <5.2.0              >=4.0.0 <5.0.0 @jupyterlab/coreutils
>=3.1.3 <3.2.0              >=2.0.0 <3.0.0 @jupyterlab/docmanager
>=3.1.3 <3.2.0              >=2.0.0 <3.0.0 @jupyterlab/filebrowser
>=6.1.3 <6.2.0              >=5.0.0 <6.0.0 @jupyterlab/services
>=3.1.3 <3.2.0              >=2.0.0 <3.0.0 @jupyterlab/settingregistry
>=3.1.3 <3.2.0              >=2.0.0 <3.0.0 @jupyterlab/ui-components


@jhgoebbert
Copy link

Thanks to @beenje the issue got fixed for jupyterlab-gitlab (https://pypi.org/project/jupyterlab-gitlab) but not jupyterlab-github.

Copy link
Member

@fcollonval fcollonval left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small comment on the new style structure.

And the index.css file should not be imported any longer in src/index.ts

https://github.com/khwj/jupyterlab-github/blob/6a25e3c02cb2032ea4e9c3c79777e6ff5c4b9057/src/index.ts#L22

@@ -3,6 +3,8 @@
| Distributed under the terms of the Modified BSD License.
|----------------------------------------------------------------------------*/

@import url('base.css');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All the styles must be moved to base.css

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this documented somewhere?

@krassowski krassowski merged commit b0dac18 into jupyterlab:master Oct 22, 2021
@welcome
Copy link

welcome bot commented Oct 22, 2021

Congrats on your first merged pull request in this project! 🎉
congrats
Thank you for contributing, we are very proud of you! ❤️

@krassowski
Copy link
Member

Thank you @khwj.

@immerrr
Copy link

immerrr commented Nov 26, 2021

Just hit this issue.

@krassowski Would it be possible to have a version including this fix published to NPM? Or is there an easy way to install this extension from source?

@krassowski
Copy link
Member

Yes, I will publish a new release tomorrow.

@krassowski
Copy link
Member

The new versions should be up on PyPI and npm. I updated readme to reflect that prebuilt extension is shipped in the PyPI version and updated references from notebook to jupyter-server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ValueError: The extension "@jupyterlab/github" does not yet support the current version of JupyterLab.
8 participants