-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Image block: Add missing space between sentences #75142
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
base: trunk
Are you sure you want to change the base?
Conversation
This has only gone unnoticed because the second sentence starts with opening parenthesis, which is enough to let the browser break the line at that point. However, I'd argue this still needs fixing: - It could break in locales in which the string is rendered with a different glyph that doesn't allow the line to break. - It can (and has) set the wrong precedent.
|
Size Change: +3 B (0%) Total Size: 3 MB
ℹ️ View Unchanged
|
andrewserong
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, and if I manually hack in the removal of the ( character, then the space is working as expected too:
I notice that elsewhere when using <ExternalLink> we've used createInterpolateElement to ensure spacing when injecting that component into a string of text. I don't think it's necessary here as the { ' ' } seems to work just fine, but thought I'd mention it just in case.
![]()
|
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. |
What?
In the Image block's advanced controls, under Title Attribute, add a space separating the sentence "Describe the role of this image on the page." and "(Note: many devices and browsers do not display this text.)".
This has only gone unnoticed because the second sentence starts with opening parenthesis, which is enough to let the browser break the line at that point. However, I'd argue this still needs fixing:
With no parentheses, we would see: