-
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
Subscription Text and Media #270
Comments
I think this one needs some more work:
|
Thanks for the inputs Ola. Ive' updated the copy. For mobile, trouble is the media/text block always puts the image on top when stacked. And if I try columns, the image is not super compatible to place inside a cover. What I did was decrease the height of the image and re-upload. Let me know what you think. If we think this image is not working, I'll try find something else that would work well with a cover. |
The cut-out is clever, but the image is not a great fit with the heading below. So yeah, I think it would be best to look at some other image options. |
Thanks! What do you think about this one? @iamtakashi @alaczek Personally I think I might want to lighten up the bg a bit more if I can without destroying the image. |
The widows in both texts jumped out to me. ("mail" and "things") We can't guarantee it to not happen all the time, but if it can be avoided most of the time by changing the copy slightly, let's do that. Also, don't we need a full stop at the end of the heading? |
Yep, agree with Takashi about the widows. Maybe shorten the heading to "Best recipes, delivered to you." or "Best recipes. Delivered to you." |
Screenshot:
Preview:
https://dotcompatterns.wordpress.com/?p=5985&preview=true
The text was updated successfully, but these errors were encountered: