Skip to content
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

Add testimonial block pattern (image in the signature). #20521

Closed

Conversation

alaczek
Copy link

@alaczek alaczek commented Feb 28, 2020

Description

Adding an alternative testimonial block pattern featuring right aligned signature with image.
See #20345

Screenshots

testimonial03

<!-- wp:group -->
<div class="wp-block-group"><div class="wp-block-group__inner-container"><!-- wp:paragraph {"fontSize":"medium"} -->
<p class="has-medium-font-size">“These guys are amazing! A very professional team that delivers excellent results every time we come for help. I highly recommend this company for your projects!”</p>
<!-- /wp:paragraph -->

<!-- wp:columns -->
<div class="wp-block-columns"><!-- wp:column {"verticalAlignment":"center","width":80} -->
<div class="wp-block-column is-vertically-aligned-center" style="flex-basis:80%"><!-- wp:paragraph {"align":"right"} -->
<p class="has-text-align-right">Jane Doe</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph {"align":"right"} -->
<p class="has-text-align-right">Creative Director</p>
<!-- /wp:paragraph --></div>
<!-- /wp:column -->

<!-- wp:column {"verticalAlignment":"center","width":20} -->
<div class="wp-block-column is-vertically-aligned-center" style="flex-basis:20%"><!-- wp:image {"sizeSlug":"large"} -->
<figure class="wp-block-image size-large"><img src="https://wpdesignhub.com/designs/basic/055854/team-member-1.png" alt="CC0 by Coffee Bean https://pixabay.com/en/users/coffeebeanworks-558718/?tab=about"/></figure>
<!-- /wp:image --></div>
<!-- /wp:column --></div>
<!-- /wp:columns --></div></div>
<!-- /wp:group -->

@gziolo
Copy link
Member

gziolo commented Nov 28, 2020

@youknowriad, you should know if we still want yo include this pattern.

@youknowriad
Copy link
Contributor

I think there has been some work after that to make the core patterns more coherent. We might not need new patterns but let's ping designers to confirm. @jasmussen @mtias @mapk

@mtias
Copy link
Member

mtias commented Nov 30, 2020

Let's close for now

@mtias mtias closed this Nov 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants