Submitted 3 months ago
i used html for structure and flex box and buttons for styling
@Danish030
Design comparison
SolutionDesign
Community feedback
- @temesgen-982Posted 3 months ago
- Semantic HTML
You probably should be using link elements
<a>
instead of buttons for the social media links since you want them to take the user into another page. - Font Size The other one is you should use the font size specified on the style guide for the paragraph elements.
Marked as helpful1 - Semantic HTML
You probably should be using link elements
- @saul-gustavoPosted 3 months ago
Hello, I saw your code and there are some things you can take into account to improve your project, in the body, you can use min-height 100vh and center the container with flexbox.
Inside the container you can use the padding property to give more padding to the content, this way you would avoid using the height and width property. You can do it, keep it up.
I would also recommend using semantic html, it will make your code look clean and easy to understand.
Marked as helpful0@Danish030Posted 3 months agothank you, i will improve and work on your recommendations @Saul-Gustavo
1
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