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

WP 6.3-beta4: not all patterns and categories listed in site editor #52608

Closed
bvlgn opened this issue Jul 13, 2023 · 8 comments · Fixed by #52633
Closed

WP 6.3-beta4: not all patterns and categories listed in site editor #52608

bvlgn opened this issue Jul 13, 2023 · 8 comments · Fixed by #52633
Labels
[Feature] Patterns A collection of blocks that can be synced (previously reusable blocks) or unsynced [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Type] Bug An existing feature does not function as intended

Comments

@bvlgn
Copy link

bvlgn commented Jul 13, 2023

Description

In de site editor only very few of the regitered patterns and categories are listed.

Step-by-step reproduction instructions

listbug

Screenshots, screen recording, code snippet

No response

Environment info

WordPress: 6.3-beta4, with no other plugins besides de beta tester pluigin.
Twenty-Twenty-Three theme

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@bvlgn
Copy link
Author

bvlgn commented Jul 13, 2023

Maybe, also have a look at the console and notice the multitude of messages that are generated when you click on the various pattern categories.

@jordesign
Copy link
Contributor

I can recreate this in testing as well. Patterns are still present in the inserter within the editor - but not in the Site Browser.

@jordesign jordesign added [Type] Bug An existing feature does not function as intended [Feature] Patterns A collection of blocks that can be synced (previously reusable blocks) or unsynced [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") labels Jul 13, 2023
@annezazu
Copy link
Contributor

@aaronrobertshaw since you've worked on this section. Adding to to do as this does feel like a departure from what was there in earlier versions/what's intended.

@ndiego
Copy link
Member

ndiego commented Jul 14, 2023

For TT3, only the three patterns that appear have Inserter set to true. So I believe this is correct. All other patterns are intentionally hidden.

If you try and theme like Frost, which has all patterns enabled, they do show up.

Image

@bvlgn
Copy link
Author

bvlgn commented Jul 14, 2023

Ah, that explains it.
But I suspect, this might be an area of confusing for other users too?

@mikachan
Copy link
Member

#52633 may also help improve this, as this will allow all patterns to be listed in the Patterns section in the editor, even if they aren't categorised. The exception would be if the pattern sets Inserter to no/false (with the help of #52629).

@ndiego
Copy link
Member

ndiego commented Jul 17, 2023

For TT3, only the three patterns that appear have Inserter set to true. So I believe this is correct. All other patterns are intentionally hidden.

I was incorrect above. The patterns that are not showing in TT3 do have Inserter: no but they are not displaying because they also do not have any assigned categories. As @mikachan mentions, #52633 and #52629 will resolve this.

@ndiego ndiego linked a pull request Jul 17, 2023 that will close this issue
@annezazu
Copy link
Contributor

Ah ha. Closing this out then in favor of #52614 Happy to reopen if there's something I'm missing but want to keep this work consolidated at this point.

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 [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Type] Bug An existing feature does not function as intended
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

5 participants