Skip to content

Fix resizing and mode switching in DockPanel#411

Merged
fcollonval merged 7 commits into
jupyterlab:mainfrom
afshin:resizer-fix
Sep 15, 2022
Merged

Fix resizing and mode switching in DockPanel#411
fcollonval merged 7 commits into
jupyterlab:mainfrom
afshin:resizer-fix

Conversation

@afshin

@afshin afshin commented Sep 14, 2022

Copy link
Copy Markdown
Member

This PR:

  • Fixes resizing in DockPanel
  • Fixes switching between single-document and multiple-document in DockLayout
  • Fixes the broken shortcut characters in the example pages
  • Adds an empty test to the docklayout.spec.ts file

This came up in the Lumino Gitter channel:

https://gitter.im/jupyterlab/lumino?at=6321ba7faa091774290be83a

@afshin afshin added the bug Something isn't working label Sep 14, 2022
@afshin afshin added this to the Lumino 2 milestone Sep 14, 2022
@afshin afshin self-assigned this Sep 14, 2022
@afshin afshin changed the title Fix resizing in DockPanel Fix resizing and mode switching in DockPanel Sep 15, 2022
Comment thread packages/widgets/src/docklayout.ts Outdated

@blink1073 blink1073 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@fcollonval fcollonval left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot for digging this one up @afshin

Let's merge and I'll do an alpha release.

@fcollonval fcollonval merged commit 4cc45f1 into jupyterlab:main Sep 15, 2022
afshin added a commit to afshin/lumino that referenced this pull request Sep 15, 2022
fcollonval pushed a commit that referenced this pull request Sep 15, 2022
* Follow-on to #411

* Un-deprecate `empty()`

* Update API
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

No open projects
Status: done

Development

Successfully merging this pull request may close these issues.

3 participants