Design comparison
SolutionDesign
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi Eligijus Linkevicius, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is and follow the sequence h1-h5
<h1 class="card-name">Equilibrium #3429</h1>
In this challenge we don't need a
media query
to change the size of the card, we can do it this way:@media only screen and (max-width: 400px) .card { /* width: 250px; */ } .card { width: 100%; max-width: 325px; }
The rest is great!
I hope it helps... 👍
Marked as helpful0
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