-
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
Grid interactivity: Make visualizer's cell UI at minimum 8x8 in size #61427
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. |
Size Change: +88 B (+0.01%) Total Size: 1.74 MB
ℹ️ View Unchanged
|
@tellthemachines: Do you think we should put this in 6.6? I'll fix the conflicts. |
I don't think it's needed for 6.6 because the problem we were trying to address with this was dragging into empty cells, which hasn't shipped yet. |
#61025 includes this. |
What?
Small fix that I've split out of #59490.
Makes the grid visualiser show empty cells as being at minimum 8x8 in size.
Why?
This makes empty rows and columns in the grid appear at least 8px wide or tall. This allows the user to drag and drop into that cell (will be necessary in the future) and more easily resize grid items to span that cell.
How?
Splits each grid visualiser cell
<div />
into a<div><div /></div>
. The outer div is positioned by the grid layout and the inner div has a min size of 8x8.This also removes the
!important
keywords from the grid visualiser CSS in favour of specificity hacks which, despite their name, are less hacky.Testing Instructions
Screenshots or screencast
Before:
After: