-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Increase dimensions of the pattern modal that appears when creating a new page #49859
Conversation
Size Change: +52 B (0%) Total Size: 1.37 MB
ℹ️ View Unchanged
|
I personally think this is good. No reason not to let you have space. Perhaps a teensy bit smaller? I wonder if we couldn't get a cleaner grid if instead of percentage widths, you just set width and heights to 100%, then added an inner padding instead? Something like 72px, that range? |
The dimensions are the same as the add-template modal. I think it makes sense for them to be equal, so we can try different sizes together once this is merged.
I'm struggling to picture what you mean 🤔 feel free to push directly to the branch if you'd like to try something. |
I'm ashamed to ask this question, I should know better: where do you see the "new page" modal? When I click to add a new page, I'm not seeing it. But to describe better, right now the modal is 95% tall, and 95% wide, which means the modal will have the same aspect ratio as the viewport. Looking at the code, it looks like this is actually what the mobile breakpoint does, more or less:
I guess I'm suggesting doing something similar for larger breakpoints, e.g.
Mainly I just think it might feel more app-like, if the modal was always the same direction from the edge as you resized the viewport. What do you think? |
Oh you're referring to the modal itself, now it makes sense. Since you mentioned grid I thought you meant the patterns :) I'll give it a try.
You need to have some patterns associated with the Post Content block. Here's an example:
|
Good intuition, this does seem better! patterns.mp4 |
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 to me. ✅
Follow-up to #49722.
What?
Increases the dimensions of the modal that displays full-page patterns when creating a new page:
patterns.mp4
I retained the 'masonry' layout because full-page patterns may not necessarily be long.
Why?
The current version feels a bit cramped:
How?
CSS.
Testing Instructions
For the modal to appear there needs to be patterns associated with the Post Content block.