P
Matt Studdert• 13,611
@mattstuddert
Posted
Awesome work on this challenge, Roy! Nitya has already given some great advice. If you're looking for some good Flexbox and Grid tutorials here are a couple of great (and free) ones by Wes Bos:
1
Design comparison
Solution retrospective
This is my first ever HTML + CSS challenge. Completed within 15 hours. I have no experience with building mobile sites yet.
Community feedback
P
Nitya Gulati• 665
@nityagulati
Posted
Hi Roy, the website looks great on desktop and tablets, nice work. Next steps would be to add CSS media queries and make the site responsive for mobile design.
Also try working with Flexbox or Grid to create the layout instead of using absolute position, that will make switching the layout for mobile design easier.
Another thing to consider, some of your class names are quite generic .div1, .div2 etc. Although perfectly fine for this project, it's a good practice to use more intuitive and meaningful names for your elements. You can look up CSS naming conventions for tips and ideas. How to name CSS classes
Keep up the good work!
1
Roybrussel• 35
@Roybrussel
Posted
@nityagulati Thank you for taking your time to review my solution! Really appreciate the feedback and I will have a read about Flexbox and Grid, didn’t know about this yet.
1
Matheus Sabino• 15
@mathesouza
Posted
Nice
1
Roybrussel• 35
@Roybrussel
Posted
@mathesouza Thank you!
0
Please log in to post a comment
Log in with GitHub
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