Used only HTML and CSS , no libraries.
Design comparison
Community feedback
- @DevibtissamPosted over 2 years ago
hey, good job on this challenge! wrap everything in your body in <main> OR use semantic tags! if you're using div right after body, then give them a role=" "; however, it's a little frowned upon to use role right after body, so try to stick with semantic tags instead. another thing related to the design in desktop devices try to make a container and give it a max-width and then center it by using margin: auto , it should look like the design given, and don't forget about adding a space on the top and also on the sides in both devices mobile & desktop
hope this helps :))
Marked as helpful0@Alejandro-Fernandez-MaciasPosted over 2 years ago@Devibtissam Thank you so much for the helpful feedback. Any constructive feedback is highly appreciated especially since Iām just starting out as a developer. Will keep on practicing and improving my code, thanks again šš½
0
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