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
Gallery: Update button rendering (Move backward, Move forward, Remove) #23471
Conversation
Size Change: +678 B (0%) Total Size: 1.13 MB
ℹ️ View Unchanged
|
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 good. The buttons do look a bit small compared to tho block toolbar.
@ellatrix I can't recall, but it think that maybe by design - at least, based on the previously defined styles. cc'ing @jasmussen |
It's not directly intentional, it's more of ensuring a decent overall balance until such a time as we can replace the gallery block with one that simply nests image blocks inside (every you'd get the mover for free). In that vein, the buttons just need to be at least 24x24 and it's fine. You could replace the big close icon with the recently merged "closeSmall", for a better balance. |
@jasmussen Thank you for your thoughts! I just checked and the buttons are 24x24. I replaced the |
Ship it! |
This update fixes the Move backward, Move forwarded, and Remove button rendering for the Gallery block.
The issue seemed to come from some newer styles overridding the padding resets of the Gallery buttons.
The solution involved increasing the specificity of the Gallery button selectors.
How has this been tested?
npm run dev
(Test in Chrome, Safari, and Firefox)
Checklist:
Resolves: #23469