@kanuos
Posted
Hi @Igor, the end product looks great! Congratulations on your first submission.
However, a few things you might want to consider :
- Text below the Heading should be of the lighter color than it is now
- A bit less padding on the card would make it look closer the to design
- Instead of using generic
div
element to wrap the card, use a semantic tag likesection
ormain
. - It's always better to specify the
object-fit
property of animg
as well as making it'sdisplay
to something other than the defaultinline
value. - Leave the default attributions in the HTML for screen readers. Hide them using CSS.
Overall, it's a pretty decent effort! Happy coding!
Marked as helpful