-
Notifications
You must be signed in to change notification settings - Fork 117
Left Aligned Call to Action with Image pattern adjustments #292
Conversation
patterns/left-aligned-cta-image.php
Outdated
<div class="wp-block-columns alignfull"><!-- wp:column {"width":"642px","style":{"spacing":{"padding":{"top":"40px","right":"40px","bottom":"40px","left":"40px"}}}} --> | ||
<!-- wp:group {"align":"wide","style":{"spacing":{"margin":{"top":"var:preset|spacing|50","bottom":"var:preset|spacing|50"}}},"layout":{"type":"constrained","wideSize":"","contentSize":"1280px"}} --> | ||
<div class="wp-block-group alignwide" style="margin-top:var(--wp--preset--spacing--50);margin-bottom:var(--wp--preset--spacing--50)"><!-- wp:columns {"align":"full"} --> | ||
<div class="wp-block-columns alignfull"><!-- wp:column {"style":{"spacing":{"padding":{"top":"40px","right":"40px","bottom":"40px","left":"0px"}}}} --> | ||
<div class="wp-block-column" style="padding-top:40px;padding-right:40px;padding-bottom:40px;padding-left:40px;flex-basis:642px"><!-- wp:heading --> |
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.
When I add the pattern I am getting a block validation error, because the padding-left and the flex basis are incorrect:
logger.js:17 Block validation: Block validation failed for `core/column` (Object).
Content generated by `save` function:
<div class="wp-block-column" style="padding-top:40px;padding-right:40px;padding-bottom:40px;padding-left:0px"></div>
Content retrieved from post body:
<div class="wp-block-column" style="padding-top:40px;padding-right:40px;padding-bottom:40px;padding-left:40px;flex-basis:642px">
</div>
(anonymous) @ logger.js:17
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.
Thanks for looking into it Carolina 😄 - I adjusted things to fix the issue.
Can you please rebase this and leave the changes to the checkmarks out of this PR? let's address that bit in #150 |
Sure, I was about to write the same thing. The goal of this PR was to cover #247 I'll rebase and leave the custom list styles implementation to Aurooba |
I made a comment on the other PR to make sure we remember to keep the visual adjustments for the new markup |
Description
Patterns tweaks from Beatriz.
Removed padding left from first column to match designs.
Fixes #247
Screenshots
Screen.Recording.2023-09-08.at.17.58.49.mov