BaseControl: add text-wrap: pretty#79112
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Size Change: +13 B (0%) Total Size: 8.59 MB 📦 View Changed
|
juanfra
left a comment
There was a problem hiding this comment.
Thanks for the PR. LGTM!
Do we want to make a pass on other help text styles to unify this? If so, can definitely be separate from this.
|
Thanks for the review. I feel like this should cover most bases, what am I forgetting? |
|
Flaky tests detected in e5884fe. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/27345756500
|
|
Should we update |
True, it covers most cases. We could consider updating Field, I can open a new PR for that if we want to expedite this one. |
What?
Pretty tiny PR to address an issue with BaseControl. The icon block revealed that there is a potential for typographic widows in the help text:

This PR adds
text-wrap: pretty;to address that:Why?
This improves legibility.
Testing Instructions
Insert the Icon block, pick an icon, and observe the help text in the inspector, it should not have a typographic widow.
Use of AI Tools
I used search, little else.