Fix: block-mover button tooltip position#77588
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. |
|
Apologies for the feedback after the merge, but could we change the position only when the Recording.2026-04-23.132817.mp4 |
|
Good point, I had forgotten to check that variation. @DarkMatter-999 , can you work on a PR for this tweak, too? |
|
Oops. Thanks for the feedback. |
What?
Part of #73785
This PR adjusts the tooltip positioning for the block mover buttons. It ensures the "Move Down" button displays its tooltip below the button, while the "Move Up" button (and any others) displays its tooltip above.
Why?
Previously, the tooltips for the block mover controls could overlap other UI elements or the buttons themselves depending on the context. By explicitly setting the tooltip position to 'top' for the up button and 'bottom' for the down button, we provide better visual clarity and prevent the tooltip from obscuring the adjacent mover control.
How?
Modified the
BlockMoverButtoncomponent to dynamically set thetooltipPositionprop based on thedirectionprop.Testing Instructions
Screenshots or screencast
Before:
block-mover-tooltip.mov
After:
block-mover-tooltip-fix.mov
Use of AI Tools
I used Gemini 3 Flash to help format this PR description and summarize the logic implementation based on my git diff.
cc: @ciampo, @mirka