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

Improve title on accordion label #406

Merged
merged 1 commit into from
Sep 14, 2022
Merged

Improve title on accordion label #406

merged 1 commit into from
Sep 14, 2022

Conversation

fcollonval
Copy link
Member

@fcollonval fcollonval commented Sep 14, 2022

The label may be truncated in the UI if it is too long due to CSS styling. The title is expected to be the fallback to the full information in such case.

@fcollonval fcollonval added the enhancement New feature or request label Sep 14, 2022
@fcollonval fcollonval added this to the 1.x milestone Sep 14, 2022
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.

Thanks!

@fcollonval fcollonval merged commit 4964661 into main Sep 14, 2022
@fcollonval fcollonval deleted the fcollonval-patch-1 branch September 14, 2022 11:16
@fcollonval
Copy link
Member Author

@meeseeksdev please backport to 1.x

meeseeksmachine pushed a commit to meeseeksmachine/lumino that referenced this pull request Sep 14, 2022
fcollonval added a commit that referenced this pull request Sep 14, 2022
@vidartf
Copy link
Member

vidartf commented Sep 14, 2022

For those of us following along on the repo: What issue does this fix? Why is it important to have the title fall back to the label if the caption is not present? Please make some notes in either the PR or the commit so that the rest of us can understand what is going on, or if someone in the future is wondering why this change was made, they can understand.

fcollonval added a commit to fcollonval/benchmarks that referenced this pull request Sep 19, 2022
fcollonval added a commit to jupyterlab/benchmarks that referenced this pull request Sep 22, 2022
* Fix command to run memory-leak test

* Acknowledge confirmation dialog

* Support new tab selector following jupyterlab/lumino#406

* Add timeout to let Plotly figures resize

* Skip motion for now as it is not a motion but a delete + insert action
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 15, 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