Design comparison
SolutionDesign
Community feedback
- @azizp128Posted about 2 years ago
Hi Fazza, congratulations for your solution!
Your solution seems fine, but you have a problem with the responsive display. Your solution is good for desktop display but will change completely when the screen is resized. Maybe you can learn more about responsive displays. In addition, your CSS code also has many same properties that are written repeatedly, you can use the grouping technique to select HTML elements with the same style definitions as below.
.fa-facebook, .fa-twitter, .fa-instagram{ border-radius: 50%; border-color: white; border-style: solid; border-width: 1px; padding: 7px 10px 5px 10px; margin-right: 10px; }
Marked as helpful0 - @erwinfazzaPosted about 2 years ago
ya, thanks for feedback, i'm very beginner for programing
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