Skip to content
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

Extension is disabled in Notebook 7 #1284

Open
matthewgasbarro opened this issue Oct 25, 2023 · 5 comments
Open

Extension is disabled in Notebook 7 #1284

matthewgasbarro opened this issue Oct 25, 2023 · 5 comments
Labels

Comments

@matthewgasbarro
Copy link

matthewgasbarro commented Oct 25, 2023

Description

When opening a notebook in Notebook 7 (not nbclassic), the git extension is greyed out in the notebook toolbar and the console contains errors that the @jupyterlab/git:clone and @jupyterlab/git:plugin plugins failed to activate

Reproduce

  1. Install jupyterlab-git==0.5.0.a.0
  2. Open a notebook in Notebook 7
  3. Open browser console to see errors
  4. See git button in toolbar greyed out

Expected behavior

Either the git extension is supported in Notebook 7 or it should be properly excluded such that there are no console errors and greyed out functionality.

Context

jupyterlab-git==0.5.0.a.0
notebook 7.0.4
jupyter_server 2.9.1

image

image

@welcome
Copy link

welcome bot commented Oct 25, 2023

Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please try to follow the issue template as it helps other other community members to contribute more effectively.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also an intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@fcollonval
Copy link
Member

Hey @matthewgasbarro thanks for trying this extension with notebook 7.

  • The git button is a feature from nbdime; I opened an issue there
  • We can certainly makes this extension compatible with notebook 7. My questions are:
    • Where should the git panel (displayed in a sidebar in Lab4) be placed? The tab panel with the filebrowser or a sidebar with the notebook/file document editors?
    • Does the clone button makes also sense in the filebrowser toolbar?

@matthewgasbarro
Copy link
Author

Hey @matthewgasbarro thanks for trying this extension with notebook 7.

  • The git button is a feature from nbdime; I opened an issue there

  • We can certainly makes this extension compatible with notebook 7. My questions are:

    • Where should the git panel (displayed in a sidebar in Lab4) be placed? The tab panel with the filebrowser or a sidebar with the notebook/file document editors?
    • Does the clone button makes also sense in the filebrowser toolbar?

Hi Frederic,

Are the plugins in the console error from nbdime too or the jupyterlab-git extension? Also, I noticed the Git menu in the toolbar just shows Help as an item with a side arrow that does not open. Not sure if that is also a nbdime issue.
image

If you decide to support Notebook, then I think the sidebar makes sense, similar to the debugger in Notebook 7:

image

Thanks!
Matt

@fcollonval
Copy link
Member

Thanks for the inputs.

Are the plugins in the console error from nbdime too or the jupyterlab-git extension?

No they come from the jupyterlab-git extension.

@matthewgasbarro
Copy link
Author

Wanted to follow up on this issue. Is there plans to support Git in Notebook 7? If not, or at least until such feature is ready, can the git and nbdiff toolbar icons be disabled (removed) in Notebook 7. Also the Git menu should be removed. It just contains a Help submenu that has no contents. See image from initial post.

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

No branches or pull requests

2 participants