-
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
Library view: Adjust controls for all template parts and reusable blocks #51591
Comments
On isolation for just this case, I think it can work. I think we need to still find a design that works for sections that do not have icons, because we'll have a lot. Though important to consider #51144 which looks likely to land. |
Perhaps just an empty space would work fine; though not if the whole group does not have icons. |
I mentioned this in another issue as well. As the > gives an indication of drilling down or previewing. |
Something I always wanted to ask: why isn't there a "main" template part between header and footer? It could contain post-content and post-meta and simplify things. |
Yea I was thinking that too. The extenalLink feels a bit odd - as it's not a new tab. But what about if we had a small arrow up right? Or is that the same thing ha? |
I don't think the cogs work, and furthermore once we get extensibility here, there are going to be tons of legitimate use cases for not having an icon, so I really think we need a design where not having a main icon on the left works. |
Yep, I think that can work, perhaps the question is whether it would look weirder if the text was indented past the missing icon or not. Main challenge is that the icon footprint doesn't perfectly align with the text, so we might want to have at least, say, a 4px left padding for text-only items so it feels like it kinda lines up. |
Right, another solution is to adjust the left padding of the icon button to optically match. |
I don't think we should do that tbh:
|
@WordPress/gutenberg-design What do you think about using the same treatment as the MenuItems do in the Library panel for the Reusable / Template Parts controls?
*Or even prepend "All" to both, i.e. "All template parts"
The text was updated successfully, but these errors were encountered: