Skip to content

theme.json: Unify property order #70033

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

Merged
merged 1 commit into from
May 1, 2025
Merged

theme.json: Unify property order #70033

merged 1 commit into from
May 1, 2025

Conversation

t-hamano
Copy link
Contributor

@t-hamano t-hamano commented May 1, 2025

What?

This PR simply changes the property order.

From:

  • styles.elements
  • styles.spacing
  • styles.blocks

To:

  • styles.blocks
  • styles.elements
  • styles.spacing

Why?

When this file is updated, the equal core file needs to be updated manually. However, there are slight differences in the order of the properties, making it difficult to see the differences.

Testing Instructions

This has no effect on the behavior of theme.json.

@t-hamano t-hamano added [Type] Code Quality Issues or PRs that relate to code quality No Core Sync Required Indicates that any changes do not need to be synced to WordPress Core labels May 1, 2025
@t-hamano t-hamano self-assigned this May 1, 2025
@t-hamano t-hamano requested a review from spacedmonkey as a code owner May 1, 2025 02:27
@t-hamano t-hamano merged commit 584cbce into trunk May 1, 2025
68 checks passed
@t-hamano t-hamano deleted the theme-json-field-order branch May 1, 2025 03:15
@github-actions github-actions bot added this to the Gutenberg 20.8 milestone May 1, 2025
chriszarate pushed a commit to chriszarate/gutenberg that referenced this pull request Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Core Sync Required Indicates that any changes do not need to be synced to WordPress Core [Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant