-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Fix alignment of the "Delete template" button in the post editor #37450
Conversation
Size Change: +13 B (0%) Total Size: 1.13 MB
ℹ️ View Unchanged
|
Thanks for fixing this, @jameskoster. P.S. I also see the overflow-y. It would be nice to fix this issue as well. Maybe a separate PR or here. Screenshot |
@jasmussen it crossed my mind, but I wasn't aware of the context for that @Mamaduka good spot! Seems to be an issue with custom negative margins in this particular UI. I'll fix that now. |
If we can, we could definitely fix (remove) the rules at the source. It's a modicum of more testing but it's a nice code quality win. |
The min-width seems to be a result of #34532 I'm not sure we should remove that? 🤔 |
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.
Looks great 👍
Closing in favor of #37442 |
Just a tiny patch this one. The
min-width
applied to menu items was causing an odd text alignment issue in the Delete Template button.