-
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
Update Pattern block category and add documentation #36144
Conversation
Co-authored-by: Carolina Nymark <[email protected]>
Maybe add a link to the internationalization guidelines? |
Added a few more comments. -I believe the change to the block category should also be backported to core? |
Co-authored-by: Carolina Nymark <[email protected]>
Co-authored-by: Carolina Nymark <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work here! 💯
'<a href="' . esc_url( __( 'https://wordpress.org', 'myblocktheme' ) ) . '" rel="nofollow">WordPress</a>' | ||
) . '</p> | ||
<!-- /wp:paragraph -->', | ||
'inserter' => false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we highlight this flag (inserter=>false
) that hides such patterns from the list of displayed patterns in inserter/explorer. etc...?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that would be super helpful
* Update pattern category and add documentation. * Fix typo Co-authored-by: Carolina Nymark <[email protected]> * Add link to i18n docs. * Update docs/how-to-guides/themes/block-theme-overview.md Co-authored-by: Carolina Nymark <[email protected]> * Update docs/how-to-guides/themes/block-theme-overview.md Co-authored-by: Carolina Nymark <[email protected]> Co-authored-by: Carolina Nymark <[email protected]> Co-authored-by: Ben Dwyer <[email protected]>
Description
This PR adds some documentation to the Block Theme Overview about the pattern block and how it can be used.
It also updates the block's category to
theme
, since block themes will primarily use this block inside their templates.Types of changes
Documentation
Checklist:
*.native.js
files for terms that need renaming or removal).