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

Add collapse and expand methods to AccordionPanel #321

Merged
merged 3 commits into from
Aug 1, 2022

Conversation

fcollonval
Copy link
Member

@fcollonval fcollonval commented Jul 26, 2022

Reference

Required for jupyterlab/jupyterlab#12866

Code Changes

Add API to programmatically expand or collapse a AccordionPanel widget. Before only user by clicking on the title could do that.

It is required for the extensions manager so we can show or hide the warning about installing third-party extensions depending on a settings parameter.

@fcollonval fcollonval added the enhancement New feature or request label Jul 26, 2022
@afshin afshin added this to the Lumino 2 milestone Jul 27, 2022
@vidartf
Copy link
Member

vidartf commented Jul 27, 2022

Would you mind writing a little bit more about why this is a good feature to have, and what it enables? It is hard to quickly parse that from the Lab PR (it has a diff of +2,238 −2,776 lines).

Also, I see this was added to the Lumino 2 milestone, but I do not think this is backwards incompatible?

@fcollonval
Copy link
Member Author

Would you mind writing a little bit more about why this is a good feature to have, and what it enables? It is hard to quickly parse that from the Lab PR (it has a diff of +2,238 −2,776 lines).

Done

Also, I see this was added to the Lumino 2 milestone, but I do not think this is backwards incompatible?

Indeed this is compatible with the current version extending the API.

@fcollonval fcollonval removed this from the Lumino 2 milestone Jul 28, 2022
@fcollonval fcollonval requested a review from afshin August 1, 2022 11:23
Copy link
Member

@afshin afshin left a comment

Choose a reason for hiding this comment

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

Thank you!

@afshin afshin merged commit 66f454b into jupyterlab:main Aug 1, 2022
@afshin
Copy link
Member

afshin commented Aug 1, 2022

@meeseeksdev please backport to 1.x

meeseeksmachine pushed a commit to meeseeksmachine/lumino that referenced this pull request Aug 1, 2022
afshin added a commit that referenced this pull request Aug 1, 2022
…on-1.x

Backport PR #321 on branch 1.x (Add `collapse` and `expand` methods to `AccordionPanel`)
@fcollonval fcollonval deleted the ft/collapse-expand-ft branch August 1, 2022 13:54
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants