-
Notifications
You must be signed in to change notification settings - Fork 2
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
Comments
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!" |
Certainly. I'll make another variation too. |
Perhaps optical but the space above and bellow the heading is uneven. |
The difference is subtle, I've adjusted line-height further to make the spaces appear as close as possible. 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. |
The text was updated successfully, but these errors were encountered: