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

Docs: Fix a few minor issues in Create block theme documentation page #36211

Closed
wants to merge 1 commit into from

Conversation

audrasjb
Copy link
Contributor

@audrasjb audrasjb commented Nov 4, 2021

Description

This PR aims to fix a few minor issues in the Create block theme documentation page:

  • replace backticks with <code> in callouts since backticks are not interpreted when using inside HTML bits
  • remove some unwanted line breaks
  • fix some missing list declarations
  • fix an issue in the name of a parameter

How has this been tested?

N/A.

Screenshots

N/A.

Types of changes

Bug fix (non-breaking change which fixes an issue)

This PR aims to fix a few minor issues in the `Create block theme` documentation page.
@carolinan
Copy link
Contributor

carolinan commented Nov 4, 2021

I think this update needs to wait until we know for sure how the templates and template parts will be accessed.

For example it still says:

There are several ways to create templates and template parts:

Manually, by creating HTML files containing block markup.
Using the site editor.
Using the template editing mode in the block editor.
The fourth way is temporary and involves going to the Appearance menu > Templates, and is not recommended because of its limitations.

https://developer.wordpress.org/block-editor/how-to-guides/themes/create-block-theme/#creating-the-templates-and-template-parts

This was accurate at the time but is likely to change in the next days or hours. :)

@audrasjb
Copy link
Contributor Author

audrasjb commented Nov 4, 2021

Oh yes, got it. Then let's wait a few days before fixing the rest of the items :)

@carolinan
Copy link
Contributor

Closing in favor of #37655 that has been updated to include some of the fixes from this PR.

@carolinan carolinan closed this Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants