Design comparison
Community feedback
- @HassiaiPosted over 1 year ago
Replace <div class="attribution"> with the footer tag, <p class="title"> with <h1> to fix the accessibility issues and add the alt attribute
alt=" "
to the img tag .For a responsive content, replace the width in .container with a max-width.
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful0 - @mrsb2Posted over 1 year ago
You can use flex and flex-direction to center divs instead of left,top and transform:translate, and you can use :hover to do that effect without the need to use a lot of divs... at least that's how i would do it...oh and border-radius somewhere is 10px and somewhere is 20px you might want to make sure that are the same... hope it helps...
0
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