Huddle landing page with a single introductory section (💬 HTML & CSS)
Design comparison
Solution retrospective
Hello 👋!
Quite easy challenge, no difficulty encountered here.
Thank you for your feedbacks 🙏 !
Community feedback
- @DudeldupsPosted over 1 year ago
Hello 👋
I have some tips for you:
-
You should always check your solutions for all screen sizes. There is a major flaw with your body element, since you declare a
max-width
on it, your page has a horizontal scroll bar. -
There's no need to remove the mobile image after 375px. The mobile view can easily go beyond 500px (especially in this challenge, because the "desktop view" is quite wide
-
The way you wrote your CSS is not a mobile-first approach. You're better off declaring all the mobile CSS rules and then use for example a media query to adjust the site for larger screens. This way, you can just "add" rules for larger screens but you have a base design that you can build on.
Hope this helps, happy coding 👾
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