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

added dynamic functionality to the text spacing #14561

Merged
merged 1 commit into from
May 20, 2023

Conversation

j264415
Copy link
Contributor

@j264415 j264415 commented May 17, 2023

References

#9684 Adjustable Text Spacing (Low priority)

Fixes #12999

Code changes

Changed height in CSS of Launcher file on the: LauncherCard, LauncherCard-label and LauncherCard-label p to "min-height" to allow for text spacing to be dynamic.

User-facing changes

When the issue on #9684 is reproduced, the code changes allow for the full text within the Launcher cards to fully be shown no matter the text spacing or font, as the launcher card will automatically adjust to the text format.

To test changes, inspect page and go to where the launcher card titles are written and add extra text with spacing within the class="jp-LauncherCard-label" <p>...</p> tag.

Backwards-incompatible changes

None

@jupyterlab-probot
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@welcome
Copy link

welcome bot commented May 17, 2023

Thanks for submitting your first pull request! You are awesome! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please make sure you followed the pull request template, as this will help us review your contribution more quickly.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also a intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@github-actions github-actions bot added Design System CSS pkg:launcher tag:CSS For general CSS related issues and pecadilloes labels May 17, 2023
@j264415
Copy link
Contributor Author

j264415 commented May 17, 2023

@manfromjupyter Please review changes.

@krassowski
Copy link
Member

krassowski commented May 20, 2023

I tested it locally it looks good!

Sorry, about an earlier comment that I since deleted - it was meant to go here #14560 (comment). I will force-push to remove the last commit which is not relevant here.

Copy link
Member

@krassowski krassowski 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 @j264415. This PR also fixes #12999 (I updated the description to link them).

My understanding is that while this addresses the accessibility issues when user manually overrides text spacing, #9684 could be further addressed by exposing a user-configurable spacing (hence we should not close #9684 yet).

@krassowski krassowski merged commit 3b0641b into jupyterlab:main May 20, 2023
@welcome
Copy link

welcome bot commented May 20, 2023

Congrats on your first merged pull request in this project! 🎉
congrats
Thank you for contributing, we are very proud of you! ❤️

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Show Contextual Help" launcher button label is truncated on Linux
2 participants