Responsive design using HTML5 & CSS3 (Flexbox)

Solution retrospective
second attempt at this challenge, i'm opened for feedback if anything needs improvement
Please log in to post a comment
Log in with GitHubCommunity feedback
- @grace-snow
In your html, get everything into meaningful elements. Those stats should be in a list of 3 items.
Your class names are a bit strange/confusing in format. Maybe read up on naming conventions like BEM to see if that helps you
In css, you shouldn’t need to put explicit heights on things. That’s a bad practice to get into - what if an editor adds more text or a user zooms their font size. Let elements grow by using min- instead (if a height is needed at all)
When using flexbox, you’ll get more responsive results by using flex properties like -basis on the flex children instead of widths. Not essential but worth trying
Last thing - I’m seeing desktop view on my mobile landscape. Probably because everything’s so small(?)
Good luck
- @grace-snow
This is all teeny tiny so I can’t read it (viewing on mobile)
- @Smita-14
Your website shows mobile screen size even on the desktop screen size.
Join 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