-
Notifications
You must be signed in to change notification settings - Fork 27
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
Content updates from Page Editor #501
Conversation
@StevenDufresne I think you made this change, can you add some context to this PR? |
Yep, follow-up to: #494 (comment) |
cc45996
to
276e772
Compare
<li class="has-white-color has-text-color has-link-color"><?php _e( 'Connect with WordPress across social media platforms: <a href="https://www.tumblr.com/wordpress">Tumblr</a>, <a href="https://x.com/wordpress">X/Twitter</a>, <a href="https://www.facebook.com/WordPress/">Facebook</a>, <a href="https://www.instagram.com/wordpress/">Instagram</a>, <a href="https://www.linkedin.com/company/wordpress/">Linkedin</a>, and <a href="https://www.youtube.com/wordpress">YouTube</a>.', 'wporg' ); ?></li> | ||
<li class="has-white-color has-text-color has-link-color"><?php _e( 'Connect with WordPress across social media platforms: <a href="https://www.tumblr.com/wordpress">Tumblr</a>, <a href="https://x.com/wordpress">X/Twitter</a>, <a href="https://www.facebook.com/WordPress/">Facebook</a>, <a href="https://www.instagram.com/wordpress/">Instagram</a>, <a href="https://www.linkedin.com/company/wordpress/">Linkedin</a>, and <a href="https://www.youtube.com/wordpress">YouTube</a>.', 'wporg' ); ?></li> |
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.
@ndiego Was replacing these spaces with
intentional? It creates a string change so I just want to check.
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.
No it wasn't 🤦♂️ That must have been done automatically. I only intended to add the padding.
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.
okay, I can remove it.
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.
Oh strange, it's still a string change— I think those spaces were already using the non-breaking space, just not encoded. In any case, I've updated it to use regular spaces which fixes some awkward wrapping on mid-sized screens.
7ddc346
to
b22d243
Compare
For clarity, these are the changes in this PR:
|
b22d243
to
a707a9a
Compare
The following pages have been updated in the Page Editor.
Screenshots: https://github.com/WordPress/wporg-main-2022/actions/runs/11018010318/artifacts/1972707741
Please review, merge, and deploy.