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

Update Pattern block category and add documentation #36144

Merged
merged 5 commits into from
Nov 26, 2021

Conversation

jffng
Copy link
Contributor

@jffng jffng commented Nov 2, 2021

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:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • I've tested my changes with keyboard and screen readers.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR (please manually search all *.native.js files for terms that need renaming or removal).

@jffng jffng added [Type] Developer Documentation Documentation for developers [Block] Pattern Affects the Patterns Block labels Nov 2, 2021
@jffng jffng requested a review from ajitbohra as a code owner November 2, 2021 14:02
Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>
@carolinan
Copy link
Contributor

Maybe add a link to the internationalization guidelines?

@carolinan carolinan added the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 17, 2021
@carolinan
Copy link
Contributor

Added a few more comments. -I believe the change to the block category should also be backported to core?

Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>
@scruffian scruffian requested a review from mkaz as a code owner November 19, 2021 10:44
Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>
Copy link
Contributor

@scruffian scruffian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@scruffian scruffian merged commit 6fd4b21 into trunk Nov 26, 2021
@scruffian scruffian deleted the update/pattern-block-category branch November 26, 2021 13:16
@github-actions github-actions bot added this to the Gutenberg 12.1 milestone Nov 26, 2021
Copy link
Contributor

@ntsekouras ntsekouras left a 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
Copy link
Contributor

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...?

Copy link
Contributor

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

@noisysocks noisysocks removed the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 28, 2021
noisysocks pushed a commit that referenced this pull request Nov 29, 2021
* Update pattern category and add documentation.

* Fix typo

Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>

* Add link to i18n docs.

* Update docs/how-to-guides/themes/block-theme-overview.md

Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>

* Update docs/how-to-guides/themes/block-theme-overview.md

Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>

Co-authored-by: Carolina Nymark <hi@themesbycarolina.com>
Co-authored-by: Ben Dwyer <ben@scruffian.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Pattern Affects the Patterns Block [Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants