-
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
Relocate template name setting into the main toolbar #23871
Comments
@mtias Do you mean the block toolbar like this Or the top toolbar like this: I have been exploring the naming integration with the block toolbar as the reusable block UI feels pretty clunky. Something like this perhaps: |
I like that, @MichaelArestad! What happens if I click to add a block without adding a name? Do I just see the "Untitled template part" in the toolbar? Or am I not allowed to add a block until I create the name? |
@mapk Exactly. You would just see something like "Untitled template part". |
How do we think this relates to #22064 and vice-versa? We have some designs with a div saying "changes apply everywhere the block is used", which seem like it could fit well with some of these looks |
@noahshrader Thanks for mocking that up. That's definitely a step in the right direction. I believe there is work being done on the link UI that uses a very similar type of form that we could perhaps use that could get us pretty close to what is in the mockup above. As for @22064, it sounds like starting with a border and potentially snackbar will be a good starting point there which doesn't conflict with this design change. |
Thanks for the feedback! Do you think we should block #24450 for that change, or can we go ahead and merge this less polished version of the title-in-toolbar as an interim step? |
@noahtallen Definitely don't block it. Let's just follow up with the polished version when we can. |
Looking at the designs in this comment: #24990 (comment), is that the interaction we want with respect to renaming template parts? clicking "rename template part" in that screenshot would then do that sliding animation thing? |
@noahtallen I'm not sure just yet. @jameskoster did you want to chime in here? |
My thoughts here are twofold:
The second point feels particularly important to me. That said, where this menu appears isn't clear right now. It might be accessed via the block toolbar, but if we go ahead with #25085 then it might be accessed by clicking the template part name in the top bar instead. It may even be that the sidebar is the appropriate location for these items. There is a discussion to be had around whether these actions should appear in multiple places, and if they do, should they be presented identically? It's hard to know what happens when you click "rename template part" without answering those questions first. Indeed the question itself might be moot – if we go the top bar route then the name input might be revealed in the menu, as suggested here. |
Closing as this is done. |
The name for a template part should be included in the toolbar itself:
The text was updated successfully, but these errors were encountered: