Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Add optional argument for changing title of JupyterLab tab. #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

litghost
Copy link

No description provided.

Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
@litghost litghost changed the title WIP: Add title argument for displaying in JupyterLab tab. Add title argument for displaying in JupyterLab tab. Apr 23, 2021
@litghost litghost changed the title Add title argument for displaying in JupyterLab tab. Add optional argument for changing title of JupyterLab tab. Apr 23, 2021
@litghost litghost mentioned this pull request Oct 21, 2021
@@ -149,6 +149,7 @@ def alive():
def run_server(
self,
mode=None, width="100%", height=650, inline_exceptions=None,
jupyterlab_title="",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@litghost apologies for losing track of this PR for so long. If you're still interested in this, it's a great start, just two comments:

  • we can call this kwarg just title, since we already have examples of kwargs here (width and height) that apply only to one mode without including the mode in the name.
  • let's add this to the docstring below, and to the changelog.

If you've moved on and don't want to come back to this, no worries, I'll get this across the finish line.

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

Successfully merging this pull request may close these issues.

None yet

2 participants