@kenreibman
Posted
Great submission!
Your desktop layout looks great, and you did a great job making it responsive as well.
You should add a text-transform: uppercase to your header h1 to match the original design.
For your mobile layout, I initially though about limiting the width like you did with a max-width but I came to a conclusion that it looks a lot better when the content fills the entire mobile viewport width. Although it is your design choice, I believe mobile users would have an easier time reading content if it took their entire width.
Marked as helpful
@muben88
Posted
@kenreibman thank you for the feedback! much appreciated! Yeah, I had the same idea about mobile design, and I totally agree with you that as a user, the content should fill the whole screen. I will try to make it. Happy coding!
