-
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
Enhance the Placeholder to better scale and adapt #18800
Comments
The Add… button could open a dropdown similar to the Replace button proposed in #16200. |
I think we should be as fluid as we can, with it growing. However I think we can have 3 to start as fixed points:
I actually don't see the icon as important here and happy if it goes. I have similar feelings over title and description if they become a call to action of 'add your image' for example.
I think that relates to my suggestion above of a smaller phone. I think it should clear as much out of way as possible.
I would actually like placeholders to move back to just being that. The changes there would be for example dragging into the interface (I think showing a hit area interaction is still useful feedback).
How minimal can we make them? Let's get them as minimal as possible! The worst we have right now are button stacks (3 in places), we don't need that and things like the upload icon aren't giving ups anything here. |
I think we actually need to go way smaller than that, so as to accommodate something like the coblocks gallery. For example a phone query would be around 320px, which should be enough to fit three buttons. But on a phone with the coblocks gallery, each of the placeholders should probably be reduced to a single button, i.e. have a query that is < 160px. In #18745, I used these sizes:
So those three happen below the phone breakpoint.
Solid point. This needs a visual for what happens when you drop an image on a placeholder — this is an interaction that could need a love in what we have today. @pablohoneyhoney do you have any thoughts here? |
Agree, reducing the elements of the interface to its most relevant and primary actions lowers the cognitive process. Reducing copy helps, even with localization. |
I'm actually going to close this one as the primary purpose has been achieved with element queries in the placeholder component. Additional enhancements can be ticketed separately. |
The "Placeholder", or the setup state of the block, is one of the most important pieces of the block editor. It serves to:
The component has been with us from the start. It has received many new features, but it has not been revisited in context of other features that were added, such as nested blocks or narrow slots as part of other compound blocks. In some cases it does not scale down beyond being a top level block:
This pull request is a first attempt at improving the scale of the placeholder: #18745:
But some of the design decisions made in that PR could use additional conversation and insight, and it could perhaps be informed by some of the efforts being explored in #18667. Some questions to ponder:
Collage Gallery:
Mockups
Building off of mockups in #18667, here are some ideas to help the discussion along:
The base placeholder is vastly reduced to a single title, no icons, and a single primary button. This keeps it helpful for when you need quick access to any of those actions in large breakpoints, but keeps the appearance simple, which matters for templates that might include multiple placeholders.
When the placeholder is selected, it's focused and the block toolbar is present:
By left-aligning content, (right aligned in RTL languages), we can provide a natural balance to additional controls such as colors in the setup state:
This left aligned pattern can then scale to the toolbar itself:
In very narrow context, we reduce the placeholder to a single word and button:
What does the button do, then? Does it expand the placeholder to full width in a modal/light-boxy fashion? Does it simply open a popover with options for browse or add a URL?
Your thoughts are welcome!
The text was updated successfully, but these errors were encountered: