Automattic / blockpatterns

Repo of block patterns to work with
7 stars 2 forks source link

Large Paragraph with Link #229

Closed iamtakashi closed 2 years ago

iamtakashi commented 2 years ago

dotcompatterns wordpress com__p=5669 preview=true

<!-- wp:group {"align":"full","style":{"spacing":{"padding":{"top":"7vw","bottom":"7vw"},"margin":{"top":"0px","bottom":"0px"}}},"layout":{"inherit":true}} -->
<div class="wp-block-group alignfull" style="margin-top:0px;margin-bottom:0px;padding-top:7vw;padding-bottom:7vw"><!-- wp:columns {"align":"wide"} -->
<div class="wp-block-columns alignwide"><!-- wp:column {"width":"33%"} -->
<div class="wp-block-column" style="flex-basis:33%"></div>
<!-- /wp:column -->

<!-- wp:column {"width":"67%"} -->
<div class="wp-block-column" style="flex-basis:67%"><!-- wp:paragraph {"align":"left","style":{"typography":{"lineHeight":1.4,"fontSize":"29px"}}} -->
<p class="has-text-align-left" style="font-size:29px;line-height:1.4">We are driven to change lives for the better and we do that by creating places and spaces that positively impact the people who use them and the environment that surrounds them.</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph {"style":{"typography":{"fontSize":"17px"}}} -->
<p style="font-size:17px"><a href="#">View more projects →</a></p>
<!-- /wp:paragraph --></div>
<!-- /wp:column --></div>
<!-- /wp:columns --></div>
<!-- /wp:group -->

https://dotcompatterns.wordpress.com/?p=5669&preview=true

onuro commented 2 years ago

LGTM 👍

alaczek commented 2 years ago

This pattern makes me hesitate a little - taken out of context of the whole design looks off balance.

iamtakashi commented 2 years ago

taken out of context of the whole design looks off balance.

@alaczek I hear you, but I'm not sure if all the patterns should have a "balanced" layout on their own. You're right though, I don't think this will be a popular pattern, but that's just ok?

onuro commented 2 years ago

Just thinking out loud, but maybe still keep this pattern and label it something like "offset positioned text and link"?

iamtakashi commented 2 years ago

That's an interesting suggestion, but we tend to avoid describing the aesthetics of the pattern as the name.

I don't mind not adding this to the inserter. I'll table this one for now.