NFT Preview Card

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hello @spirit-101, Congratulations on completing this challenge!

  I saw your solution preview site and I think it's already really good. Here’s some tips for you to improve it:

  Your component is perfect, but is not responsive yet, this is due the fixed width you've applied to the container.

  Look both width and max-width the main difference between these properties is that the first(width) is fixed and the second(max-width) is flexible, for example, a component with width: 340px will not grow or shrink because the size will be ever the same, but a container with max-width: 340px or min-width: 340px can grow or contract depending of the property you've set for the container. So if you want a responsive block element, never use width choose or min-width or max-width.

  To improve your component overall responsiveness, something you can do its to create a media query to save space in the pricing section to make each information in a different row. Here’s the code for this media query.

  @media (max-width: 350px) {
  .info {
      display: flex;
      gap: 10px;
      margin-bottom: 1.6rem;
      font-size: 1.3rem;
      align-items: center;
      flex-direction: column;
      justify-content: center;
  }
  }
  

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  1

  ---

  

  spirit•470

  @spirit-101Posted about 2 years ago

  @correlucas Your feedback has really been helping me improve my CSS and HTML skills. Thank you so much!

  1

  

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  @spirit-101 I'm happy to hear that, keep it up! I've left you another feedback in one of your solution, see if you like it =)

  1