Design comparison
SolutionDesign
Solution retrospective
I made the chat too big, but it looks good in 1920x1080 resolution. Any advices are welcome 😀.
Community feedback
- @matt-o-westPosted 10 months ago
Hello! Thank you for submitting your solution! I have some feedback for you:
- You need some semantic HTML in your markup, such main and h1 landmarks (https://www.w3.org/WAI/ARIA/apg/patterns/landmarks/examples/main.html). Landmarks are used to define major sections of your page instead of relying on generic elements like <div> or <span>.
- Your styling is good, but you are missing responsive styling. This can be applied easily with tailwind by using the built-in breakpoints (sm, md, lg etc.) (see docs: https://tailwindcss.com/docs/screens), or you can configure your own breakpoints via the tailwind.config file.
Your solution is very nice, great job! 🎉
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