-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Featured Image Panel: Align text and icons horizontally to avoid clipping #62842
Conversation
Size Change: +96 B (+0.01%) Total Size: 1.76 MB
ℹ️ View Unchanged
|
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. |
I'm not sure if this PR can be backported to WP6.6 now, but I'll label it just in case. |
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.
Good suggestion, updated 👍 47c13f94eb402b29316f14d31ce01050.mp4 |
…ping (#62842) * Featured Image Panel: Align text and icons horizontally to avoid clipping * Apply rounded corner to drop zone Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: jasmussen <joen@git.wordpress.org> Co-authored-by: annezazu <annezazu@git.wordpress.org>
I just cherry-picked this PR to the wp/6.6 branch to get it included in the next release: b5d75a0 |
Fixes #62841
What?
This PR prevents text and icons from being cut off when you drag an image when a featured image has not yet been set.
How?
Positions the icon and text horizontally only if an image has not yet been set.
If a featured image is set,
.editor-post-featured-image__toggle
is applied to the button, otherwise.editor-post-featured-image__preview
is applied. However, since theDropZone
is placed next to this button, theDropZone
component itself cannot know whether a featured image is set or not. Therefore, I used the:has
pseudo selector.Testing Instructions
Open the post editor.
Move the file to the "Set featured Image" button.
Screenshots or screencast
5dfac7dd424e96e6d72de8014b160860.mp4