Design comparison
Community feedback
- @beowulf1958Posted 20 days ago
Congratulations on completing this challenge. Your page looks great and is responsive. I love the way you have documented your code so well with comments. I am learning tailwindcss, so I was excited to see how you implemented it on this project.
I do have two suggestions. First, the mobile version is a bit cramped and hard to read. Perhaps if you increased
w-[80%]
on the main it would be better.Second, the picture has a purple tint to it (the accent color in the style guide). This can be accomplished by adding a background color to the image div, reducing the opacity of the 2 img, and blending the img and background color with mix-blend. It would look like this:
<div class="sm:float-right bg-[#aa5cdb] bg-blend-overlay"> <img class="h-48 w-[100%] rounded-t-lg sm:hidden opacity-70" src="images/image-header-mobile.jpg" alt="image-header-mobile" /> <img class="hidden sm:block sm:h-[411px] rounded-r-lg opacity-70" src="images/image-header-desktop.jpg" alt="image-header-desktop" /> </div> `` Now it looks just like the design! Hope this helps.
Marked as helpful0@kalebemaxPosted 20 days ago@beowulf1958 thanks for your recommendations, i really appreciate it. I will try to add to the solution.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord