Conversation
@@ -1,84 +1,26 @@ | |||
<?php |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This diff looks deceptively complex. This pattern is being replaced with the one that was previously in the footer-title-tagline-social-dark.php
file, so that's why it looks like so much has changed.
inc/block-patterns.php
Outdated
'page-about-media-left', | ||
'page-about-simple-dark', | ||
'page-about-media-right', | ||
'page-about-links', | ||
'page-about-links-dark', | ||
'page-about-big-image-and-buttons', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the file be renamed to page-about-large-image-and-buttons
for consistency?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call! I'll make that change and merge. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pending my one suggestion, this LGTM.
Our "About page with big image and buttons" pattern is showing up as a duplicate in the footer category by mistake, with the title "Footer with title, tagline, and social links". I think this was an error from duplicating files back when I was creating new patterns:
This PR removes that pattern, and tidies up the naming of the actual "Footer with title, tagline, and social links" pattern. It also renames "About page with big image and buttons" to use the word "large" instead of "big" because that's more consistent with language we're using elsewhere.
Finally, since I was editing it anyway, I moved the "About page with big image and buttons" pattern down the list so that it's next to our other "link in bio"-style patterns.