@Hassiai
Posted
add the alt attribute alt=" "
to all the img tags and replace or wrap <div class="attribution"> with the <footer> to fix the accessibility. the value the alt attribute is the description of the image.
For a responsive content which won't require a media query for this challenge, replace the percentage width value of .card-holder. with a fixed max-width value . e.g max-width: 400px
.
To center .cardholder on the page using flexbox, replace the height of the main with min-height: 100vh
, there is no need for a max-width value in the main, give it a width of 100%.
Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on CSS units Click here
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful