Design comparison
SolutionDesign
Community feedback
- @eugenederiiPosted 23 days ago
-
Heading Levels: Using <h1> for "Jessica Randall" is not ideal, as this is the user’s name and not the primary heading of the page. You should use a separate <h1> tag for the main page title, such as "Social Links Profile," outside the profile card. The user’s name can be an <h2> tag instead.
-
Links: Instead of using <div class="website">GitHub</div>, you should use <a href="#" class="website">GitHub</a> to make these clickable and semantically indicate they are links.
-
In general, yes, but it is possible to improve the names of the classes and bring out all the styles in CSS 5. -
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