Design comparison
SolutionDesign
Solution retrospective
Still struggling with the background images resizing on zoom in.
Community feedback
- @ManuGil22Posted over 2 years ago
Hey there! Pretty cool solution! Have one thing to say as feedback:
- Always use heading tags in order! You should use <h1> first and if the font its too big u can always style it with css!
<h1 class="card__name">Victor Crest</h1>
Everything else seems pretty good for me! Well done :D
0@Passenger89Posted over 2 years ago@ManuGil22 Hi Manuel. Thank you for the suggestion. I used a h2 tag deliberately with the assumption that this profile card component would be part of a larger site in a real world situation.
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