-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Trusted indicator #6393
Comments
JupyterLab shows a shield on the statusbar (but there are some issues that we need to fix notebooks often being marked as not trusted even though those should be trusted. |
Ah thanks @krassowski 👍 For some reason it has never been really visible to me. Probably because it's "down" in the UI and easy to miss. I have updated the top comment to mention that. |
The Jupyter Notebook security docs suggest that JupyterLab also makes a menu option in the File menu available for trusting an untrusted notebook. Will there be a separate PR for the new menu option? |
Good point. JupyterLab has this option in the command palette but not in the menu. While I don't see a mention of JupyterLab in that link, I see the point. I guess the problem with the docs in |
Problem
The classic notebook shows a "Trust" indicator in the top area:
RetroLab currently doesn't provide such indicator.
Proposed Solution
JupyterLab shows a shied in the status bar:
And exposes a command to trust a notebook:
We can reuse the same command, and add a new button to the
menu
area to the right of the kernel status indicator:https://github.com/jupyterlab/retrolab/blob/9010ff543538dd6b15d647dd38ce44626095db84/packages/notebook-extension/src/index.ts#L165
Additional context
This indicator can be added to the
notebook-extension
: https://github.com/jupyterlab/retrolab/blob/main/packages/notebook-extension/src/index.tsThe text was updated successfully, but these errors were encountered: