Design comparison
SolutionDesign
Solution retrospective
Adding two backgrounds on the body never worked for me on the desktop view.
Community feedback
- @darryncodesPosted almost 3 years ago
Hi Kibiwot,
Overall a great solution, well done!
- Your
.rating
boxes lose their padding at tablet and mobile view. You could also look to addmax-width
to them so they don't stretch across the whole viewport too - I noticed that you could use some semantic html like a
<main>
and<footer>
and that would clear your accessibility report - Also your closing
</body>
tag isn't in the right place, it should be at the end of the document above</html>
All the best!
Marked as helpful0 - Your
- @LeskimPosted almost 3 years ago
Nice 💯
Kudos on your first challenge 💪🏽
The h1's font-size is a bit small .... also this would be a good project to use blockquotes in your project. You can look into it here :- https://developer.mozilla.org/en-US/docs/Web/HTML/Element/blockquote I also recommend adding a Readme file on your project
From Kenya to the world 🇰🇪 🤜🏽
Marked as helpful0
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