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

Custom patterns: Patterns Library and Inserter 1:1 #51945

Closed
richtabor opened this issue Jun 26, 2023 · 7 comments
Closed

Custom patterns: Patterns Library and Inserter 1:1 #51945

richtabor opened this issue Jun 26, 2023 · 7 comments
Labels
[Feature] Patterns A collection of blocks that can be synced (previously reusable blocks) or unsynced Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.

Comments

@richtabor
Copy link
Member

It's not clear that if a pattern is synced, then it does not show up in the Patterns Inserter. In the example below, I've created three patterns — two of which are "synced". Consequently, those two are omitted from the Pattern Inserter.

If I create a pattern, I think it should be available in the Inserter, as it is within the Library, synced or not.

CleanShot 2023-06-26 at 17 11 06 CleanShot 2023-06-26 at 17 15 40

Proposal

Perhaps we can include an indicator of synced status in the preview, similar to how Openverse media has an ellipsis for more actions — or a treatment like badges in List view.

CleanShot 2023-06-26 at 17 21 46 CleanShot 2023-06-26 at 17 20 20
@richtabor richtabor added Needs Design Feedback Needs general design feedback. [Feature] Patterns A collection of blocks that can be synced (previously reusable blocks) or unsynced labels Jun 26, 2023
@richtabor richtabor changed the title Patterns: Library and Inserter 1:1 Custom patterns: Library and Inserter 1:1 Jun 26, 2023
@aaronrobertshaw
Copy link
Contributor

Thanks for writing up this issue @richtabor 👍

It might be worth highlighting that there is a current bug (with a pending fix) that prevents the correct separation of synced and unsynced patterns in the library.

With the above fix in place you get the following display in the library.

Screenshot 2023-06-27 at 3 40 23 pm

I agree that how the patterns are included in the inserter could do with some love. The original idea in #51144 was that synced patterns (formerly reusable blocks) would continue to be located where the have been until now, in their own tab under the symbol icon, to minimise disruption. The new unsynced patterns would appear under the Patterns tab given they function the same way.

Unsynced Synced
Screenshot 2023-06-27 at 3 40 47 pm Screenshot 2023-06-27 at 3 41 00 pm

@glendaviesnz might be able to point to where the original discussion occurred as it is eluding me at the moment.

Is this issue something that is a must-have for an MVP to land in 6.3?

@richtabor
Copy link
Member Author

The original idea in #51144 was that synced patterns (formerly reusable blocks) would continue to be located where the have been until now, in their own tab under the symbol icon, to minimise disruption. The new unsynced patterns would appear under the Patterns tab given they function the same way.

I was thinking that both would be available in the Pattern Inserter, while not changing the existing synced patterns location (for the time being at least).

Is this issue something that is a must-have for an MVP to land in 6.3?

I do think folks will interpret this as a bug, if you can see custom patterns in your "My patterns" category of the library, but not within the inserter.

@bgardner
Copy link

bgardner commented Jun 27, 2023

I do think folks will interpret this as a bug, if you can see custom patterns in your "My patterns" category of the library, but not within the inserter.

@richtabor Agreed. I literally sent this message to @annezazu five minutes ago:

"Good morning! I know you've been doing a lot of testing with the Library (patterns)... I have a Q, which I feel like I'm missing something. So I created a synced pattern via the library, but struggling to understand how it is actually added to a post/page. I understand that non-synced patterns are available with the inserter, but cannot for the life of me figure out how to add a synced one."

@SaxonF
Copy link
Contributor

SaxonF commented Jul 7, 2023

The separation was intentional, primarily to ease people into the transition from reusable blocks -> patterns. Currently, patterns in the patterns tab aren't synced, and the synced tab contains reusable blocks. The idea is that for 6.3 we keep that behaviour so people can get used to the name change first, and then 6.4 we start to consolidate (e.g. add categories to user created patterns, merge categories in library etc).

If we are getting constant feedback about this we can add synced patterns back in, but that involves a little more work than what's on the surface:

  • We need to highlight which patterns are synced
  • We need to handle pagination as rendering potentially thousands of reusable blocks/synced patterns (some sites do have this) will kill the client performance.

@richtabor
Copy link
Member Author

I would expect them to be in both areas while the shift is occurring.

Otherwise you don't really learn the new method for finding/inserting synced patterns (i.e. reusable blocks)—you just keep using the old method, as it's the only one available anyhow.

@jordesign jordesign added the [Type] Enhancement A suggestion for improvement. label Jul 18, 2023
@glendaviesnz
Copy link
Contributor

We also need to make sure that the quick inserter is consistent, eg. if a user adds a synced pattern called 'My Pattern' and types /My pattern it will appear in the quick inserter popup, but it won't if they add it as an unsynced pattern.

@aaronrobertshaw aaronrobertshaw moved this to Todo in Patterns Aug 3, 2023
@aaronrobertshaw aaronrobertshaw changed the title Custom patterns: Library and Inserter 1:1 Custom patterns: Patterns Library and Inserter 1:1 Aug 4, 2023
@bph bph moved this from Todo to In Progress in Patterns Aug 4, 2023
@annezazu annezazu moved this to In Progress in WordPress 6.4 Editor Tasks Aug 30, 2023
@glendaviesnz
Copy link
Contributor

This was fixed by #54007 - the synced patterns tab has now been merged into the patterns tab.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Patterns A collection of blocks that can be synced (previously reusable blocks) or unsynced Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.
Projects
No open projects
Status: Done
Status: Done
Development

No branches or pull requests

6 participants