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

Empty Theme: Add the $schema property in theme.json and rename template directories #46300

Merged
merged 1 commit into from
Dec 8, 2022

Conversation

t-hamano
Copy link
Contributor

@t-hamano t-hamano commented Dec 5, 2022

What?

This PR makes the following two changes to the Empty Theme:

  • Add the $schema property in theme.json
  • Rename directories: block-templates to templates, block-template-parts to template-parts

Why?

This may not be true for all contributors, but I activate Empty Theme when isolating the cause. I also temporarily add properties when testing the reproducibility of global style-related defects, etc. In those cases, I think it would be useful to have auto-completion and validation.

As for directory names, they have changed in WordPress 5.9. The Gutenberg plugin supports WordPress versions 5.9 and above, so renaming should not be a problem now.

Testing Instructions

  • Confirm that the template (part) is recognized by the site editor when the Empty Theme is activated.
  • Open theme.json in the code editor and confirm that auto-completion and validation are performed.

@t-hamano t-hamano marked this pull request as ready for review December 5, 2022 09:12
@t-hamano t-hamano self-assigned this Dec 5, 2022
@skorasaurus
Copy link
Member

Nice work, these changes were also made in the emptytheme repo itself so they should match what's here

Copy link
Member

@jorgefilipecosta jorgefilipecosta left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jorgefilipecosta jorgefilipecosta merged commit 0db2dde into trunk Dec 8, 2022
@jorgefilipecosta jorgefilipecosta deleted the update-empty-theme branch December 8, 2022 20:42
@github-actions github-actions bot added this to the Gutenberg 14.8 milestone Dec 8, 2022
@ryanwelcher ryanwelcher added the [Feature] Themes Questions or issues with incorporating or styling blocks in a theme. label Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Themes Questions or issues with incorporating or styling blocks in a theme.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants