All comments

• 

  RobertoBaiochi• 170

  @RobertoBaiochi

  Submitted almost 2 years ago

  A cool challenge to do, any feedback?

  

  QR Code my solution

  1

  3

  0

  

  Kapline• 140

  @TrueKapline

  Posted almost 2 years ago

  Nice work! Congratulations on completing this challenge 👏

  • You should rename your h2 to h1 since every page must contain at least one.
  • Also, you don't really need so many containers such as <div class="container">, <div class="main-container"> and <section>. You can easily move content inside <main> tag without any other sections. Like this:

  main {
      display: flex;
      justify-content: center;
      align-items: center;
      flex-direction: column;
      background-color: white;
      padding: 16px;
      box-shadow: 0px 6px 10px hsl(218deg 25% 72% / 48%);
  }
  

  1
• 

  Kapline• 140

  @TrueKapline

  Submitted almost 2 years ago

  

  Responsive flexbox product preview card

  1

  1

  0

  

  Kapline• 140

  @TrueKapline

  Posted almost 2 years ago

  Why this project looks different at screenshot ლ(¯ロ¯"ლ)

  0