Design comparison
Solution retrospective
How to put that background top and bottom exactly just like they look in design view
Community feedback
- @SamadeenPosted over 2 years ago
Hey Adam!! Cheers 🥂 on completing this challenge.. . Here are my suggestions 1.You should use <main class="social-container"> instead of <div class="social-container">.
-
Go down orderly when you are using the headings h1 down to h2 down to h3 and so on.
-
There is an HTML issue that can be fixed by setting your language attribute correct.. Kindly remove the back tic
. Regardless you did amazing.. Happy coding!!!
Marked as helpful0 -
- @Akhlak-Hossain-JimPosted over 2 years ago
Great work adam. I just saw through your code and the live preview. It looks great. Here is my feedback:
- try not to use
div
directly in the body, like<div class="social-container">
useheader
main
whatever suits best in that case. - try using CSS variables for color and stuffs.
Happy coding:)
Marked as helpful0 - try not to use
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