Skip to content
Submitted 10 months ago

Html, CSS Grid, Css Flexbox, Responsive

yvodlynpaul•150
@yvodlynpaul
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Community feedback

  • Marcos Travaglini•4,840
    @BlackpachamamePosted 10 months ago

    Good job!

    Here are some comments that may help you:

    • The <div class="container"> should be <main class="container">, i think the current main is not necessary.
    • The <div class="attribution"> should be <footer class="attribution">
    • To separate the main from the footer, you can simply place a gap: 20px in the body
    0

Please log in to post a comment

Log in with GitHub

Join 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