Skip to content
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

Site Editor Inserter(s): Prioritise Template Parts over Patterns, and indicate the difference between the two #42501

Closed
jameskoster opened this issue Jul 18, 2022 · 7 comments
Labels
[Block] Template Part Affects the Template Parts Block [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Design Feedback Needs general design feedback. Needs Design Needs design efforts. [Type] Enhancement A suggestion for improvement.

Comments

@jameskoster
Copy link
Contributor

jameskoster commented Jul 18, 2022

The quick inserter currently surfaces patterns, both by default and when you search:

Screenshot 2022-07-18 at 12 00 00

But it's not always clear whether the options (like in the screenshot above) are template parts or patterns, which can lead to folks mistakenly inserting a pattern where a template part would be preferable.

This issue can be partially fixed by #41397, but it might be best to establish some visual distinction to cover cases where patterns and template parts share a name. There are a couple ways to do this, one solution would be to simply list them separately:

Screenshot 2022-07-18 at 12 41 15

This would be similar to what you find in the current implementation of the template part replacement modal:

Screenshot 2022-07-18 at 12 05 55

Another option would be to list them together but use the template part icon to distinguish between patterns and things like headers / footers. Rough example:

Screenshot 2022-07-18 at 12 39 38

@jameskoster jameskoster added Needs Design Feedback Needs general design feedback. Needs Design Needs design efforts. [Feature] Full Site Editing [Block] Template Part Affects the Template Parts Block labels Jul 18, 2022
@DaisyOlsen
Copy link
Contributor

I can see merit in both the icon indicators and the separate sections. I'd lean toward separation because template parts and patterns are functionally different so a more obvious separation might be helpful. I also think that, in most cases, parts would be used more frequently in the site editor than patterns.

@carolinan
Copy link
Contributor

I would prefer the icon because I feel a bit overwhelmed by the size of the inserter in the example.

@jameskoster
Copy link
Contributor Author

I don't have a strong opinion, but do lean slightly towards the separation approach purely because it is consistent with other UIs that present the same content. We can explore how we might improve the experience holistically as a dedicated effort, rather than having several of different versions of essentially the same thing across the editor.

@jameskoster jameskoster changed the title In the Site Editor, the quick inserter should generally prioritise existing template parts over patterns Site Editor Inserter(s): Prioritise Template Parts over Patterns, and indicate the difference between the two Aug 25, 2022
@mtias
Copy link
Member

mtias commented Sep 10, 2022

Where did we end up with this?

@jameskoster
Copy link
Contributor Author

Seems we're looking for consensus on how to present everything. These are the options we have atm:

Screenshot 2022-09-12 at 10 10 13

Now that I think of it, another option would be: Clicking the root-level inserter engages the zoomed-out view with the main Inserter panel open. The separation of template parts / patterns is handled therein.

@annezazu annezazu added [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") and removed [Feature] Full Site Editing labels Jul 24, 2023
@jordesign jordesign added the [Type] Enhancement A suggestion for improvement. label Sep 5, 2023
@jordesign
Copy link
Contributor

I'm not seeing the pattern/part preview within the quick inserter any more - is this issue still relevant @jameskoster

@jameskoster
Copy link
Contributor Author

jameskoster commented Sep 5, 2023

It's still there, but only at the document root by the looks of things:

Screenshot 2023-09-05 at 10 02 40

Seems we're looking for consensus on how to present everything.

The Pattern management view didn't exist when we last looked at this, and I think it sets a precedent that can inform the direction here:

Screenshot 2023-09-05 at 10 05 28

It uses the icon approach above to communicate the synced nature of a given pattern. This is probably adequate on its own (no need to re-order). I'll update the OP.

Edit: It seems that when the quick inserter is set to display patterns it will only return unsynced ones (IE no template parts), even when you search. So this issue doesn't really exist any more, but the quick inserter should be updated to include synced patterns. I'll open a new issue for that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Template Part Affects the Template Parts Block [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Design Feedback Needs general design feedback. Needs Design Needs design efforts. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

6 participants