P
Andreas Remdt• 950
@andreasremdt
Posted
Hey @ChrysKoum!
You did a great job on the challenge, I like that the design looks so close. The hover transitions are nicely done. I found a couple of things that you could improve:
- Make sure to use some semantic HTML elements, such as
main
orsection
. That's also a complaint on your accessibility report. For example,div.card
could be replaced bymain.card
to indicate that this is the page's main content. - You used an
h5
for the only page title, while anh1
would be the better choice. Each page should have a main title, and then go lower from there. Since "Equilibrium #3429" is the only title, using anh1
would be much better for accessibility and semantics. - The solutions appears to have a couple of interactive elements (like links), but there are no proper HTML tags for that. For example, it's confusing that the title and author have hover effects, but I can't click on them. To fix this, you can use a
a
or abutton
for these elements. - You don't need all these
div
elements to center your card. With just a couple lines of CSS, you can make your life much easier:
<body>
<main class="card">
Your card content here...
</main>
</body>
body {
display: grid;
place-items: center;
min-height: 100vh;
}
Hope that these tips help you improve the solution. Let me know if you have any questions :-)
1