Design comparison
Community feedback
- @sammyodiagbePosted over 1 year ago
One thing I noticed is the lack of spacing between the component and the text below where you typed "Challenge by frontendmentor...." a little whitespace would be nice. Lovely design by the way
Marked as helpful0@OluwaseyifunmitanPosted over 1 year ago@sammyodiagbe thanks for the tip, I’ll keep that in mind for the next challenge
0 - @MelvinAguilarPosted over 1 year ago
Hello there 👋. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
HTML 🏷️:
- Use semantic elements such as
<main>
and<footer>
to improve accessibility and organization of your page.
CSS 🎨:
- To center the content horizontally, use
justify-content: center;
to thebody
. Also, you can remove themargin: 0 auto;
in the.container
selector.
I hope you find it useful! 😄 Above all, the solution you submitted is great!
Happy coding!
Marked as helpful0@OluwaseyifunmitanPosted over 1 year ago@MelvinAguilar oh thank you very much, I’ll make sure to implement the corrections in my next challenge.
0 - Use semantic elements such as
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