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

Constrain tabs to their source DockPanel (opt-in) #137

Merged
merged 3 commits into from
Dec 11, 2020

Conversation

piersdeseilligny
Copy link
Contributor

This pull request is a modification to this one, by jtpio. I would have committed there, but obviously don't have write access.

It makes the constraining of tabs to their source dock panel an opt-in feature, through the tabsConstrained option and property, and fixes #43.

@piersdeseilligny piersdeseilligny changed the title Constrain tabs to their source panel (opt-in) Constrain tabs to their source DockPanel (opt-in) Dec 3, 2020
@afshin
Copy link
Member

afshin commented Dec 8, 2020

@piersdeseilligny thank you for updating this logic! 🔥

This adds to the API surface area and modifies behavior, could you please add to the test suite as well to cover these new updates?

@piersdeseilligny
Copy link
Contributor Author

@afshin I unfortunately don't have much experience working with unit tests, but is this latest commit similar to what you're looking for?

Copy link
Contributor

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@blink1073 blink1073 merged commit 9962658 into jupyterlab:master Dec 11, 2020
@jtpio
Copy link
Member

jtpio commented Dec 11, 2020

Nice, thanks @piersdeseilligny for picking this up!

@blink1073
Copy link
Contributor

Released as @lumino/widgets: 1.16.1 => 1.17.0

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 18, 2022
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.

DockPanel inside a DockPanel
4 participants