@ndickers
Posted
Thank you for the review. I'm checking on that.
@ndickers
Posted
Thank you for the review. I'm checking on that.
@Hassiai
Posted
Wrap <div class="attribution"> within the footer tag to fix the accessibility issue.
To center the main on the page, add min-height:100vh; display: flex; align-items: center: justify-content: center; to the body
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
Use relative units like rem or em as unit for the padding, margin, width values and preferably rem for the font-size values, instead of using px which is an absolute unit. For more on CSS units Click here
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
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