Skip to content

Reposition widgets - #26

Closed
brichet wants to merge 4 commits into
jupyter-ai-contrib:mainfrom
brichet:reposition-widgets
Closed

brichet wants to merge 4 commits into
jupyter-ai-contrib:mainfrom
brichet:reposition-widgets

Conversation

@brichet

@brichet brichet commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Add a command to reposition widgets in the lab shell.

This PR may have to be updated if #24 goes first, or the other way around, to handle the user facing strings localization.

This command initially comes from jupyterlite-ai: https://github.com/jupyterlite/ai/blob/9ef5e207a02a2d25d22701fa20ab676ec6a87040/packages/ai/src/index.ts#L739

@brichet brichet added the enhancement New feature or request label Jun 19, 2026
@jtpio

jtpio commented Jun 22, 2026

Copy link
Copy Markdown
Member

Thanks for looking into this.

Actually wondering if this is still needed with JupyterLab 4.6 or if we could use the upstream commands instead? https://jupyterlab.readthedocs.io/en/latest/user/interface_customization.html#move-open-widgets

@brichet

brichet commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator Author

Thanks for the feedback, this command is indeed partially covered by jupyterlab/jupyterlab#17880.

But I 'm not sure that there is a way to split main area by command only currently.
jupyterlab/jupyterlab#18192 introduced it, but does not allow a widget id in args, it uses only the mouse hit test.

That said, we should fix it upstream in jupyterlab, since the command already exists.

@brichet brichet closed this Jun 22, 2026
@jtpio

jtpio commented Jun 22, 2026

Copy link
Copy Markdown
Member

That said, we should fix it upstream in jupyterlab, since the command already exists.

Right we can track it in an issue in the JupyterLab repo. Maybe this could even be released in a 4.6.x patch release if it's considered as a "bug fix".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants