WordPress / twentytwentytwo

Twenty Twenty-Two, the default WordPress theme that will launch with WordPress 5.9.
406 stars 91 forks source link

Remove extra wrappers from patterns and header templates #337

Closed kjellr closed 2 years ago

kjellr commented 2 years ago

In https://github.com/WordPress/twentytwentytwo/pull/310#pullrequestreview-842834128, I added an extra Group block wrapper to many of the theme's header and footer patterns. In many cases, these were alignfull Group blocks, wrapping other alignfull Group blocks. It was redundant, but the "layout":{"inherit":true} wrapper was required in order to ensure that the contents went full-width in the site editor and in the post/page editor.

Now that #336 has been merged, many of these empty wrappers can be removed without any effect to the patterns' appearance. All patterns should look identical, they just don't need these extra wrappers any longer.

Testing

kjellr commented 2 years ago

🙌 Awesome, thank you!