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

Call to Action #86

Closed
iamtakashi opened this issue Jul 11, 2020 · 4 comments
Closed

Call to Action #86

iamtakashi opened this issue Jul 11, 2020 · 4 comments

Comments

@iamtakashi
Copy link
Contributor

iamtakashi commented Jul 11, 2020

Pattern 96

<!-- wp:group {"align":"full","style":{"color":{"text":"#000000","background":"#ffffff"}}} -->
<div class="wp-block-group alignfull has-text-color has-background" style="background-color:#ffffff;color:#000000"><div class="wp-block-group__inner-container"><!-- wp:spacer {"height":64} -->
<div style="height:64px" aria-hidden="true" class="wp-block-spacer"></div>
<!-- /wp:spacer -->

<!-- wp:paragraph {"align":"center","style":{"typography":{"fontSize":24,"lineHeight":".9"}}} -->
<p class="has-text-align-center" style="font-size:24px;line-height:.9"><strong>Interested?</strong></p>
<!-- /wp:paragraph -->

<!-- wp:heading {"align":"center","style":{"typography":{"fontSize":59,"lineHeight":"1.15"}}} -->
<h2 class="has-text-align-center" style="font-size:59px;line-height:1.15">Book a free online consultation</h2>
<!-- /wp:heading -->

<!-- wp:buttons {"align":"center"} -->
<div class="wp-block-buttons aligncenter"><!-- wp:button {"borderRadius":50,"style":{"color":{"background":"#000000","text":"#ffffff"}}} -->
<div class="wp-block-button"><a class="wp-block-button__link has-text-color has-background" style="border-radius:50px;background-color:#000000;color:#ffffff">Contact us</a></div>
<!-- /wp:button --></div>
<!-- /wp:buttons -->

<!-- wp:spacer {"height":64} -->
<div style="height:64px" aria-hidden="true" class="wp-block-spacer"></div>
<!-- /wp:spacer --></div></div>
<!-- /wp:group -->
@ianstewart
Copy link

ianstewart commented Jul 14, 2020

I like it when I see a pattern I've already given feedback on when it was part of layout I looked at. :) 👍

What do you think about a variation just on a coloured background with different text? Just to spark a possibility for a user who might not see the white background and think, "I can change that!"

@iamtakashi
Copy link
Contributor Author

What do you think about a variation just on a coloured background with different text?

Certainly. I'll make another variation too.

@pablohoneyhoney
Copy link

Perhaps optical but the space above and bellow the heading is uneven.

@iamtakashi
Copy link
Contributor Author

The difference is subtle, I've adjusted line-height further to make the spaces appear as close as possible.

Pattern 96

To combat an issue like this, we'd need margin control for every block or spacer blocks that can be less than the current minimum of 20px. We're keeping an eye on the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants