prudect-card

Community feedback

• 

  Hanana•170

  @meantoesPosted over 1 year ago

  hi! congrats on completing the challenge⭐ I have some suggestions that might interest you,

  • wrap the whole content in the <main> tag instead of the div .container

  • use alt attribute for the perfume image

  for the css, use this trick to center the container

  • add these properties to your body element

      
      body {
          /* -- */
          display: flex;
          align-items: center;
          min-height: 100vh
      }
  
  

  • in the .container element, delete margin-top: 100px
  • Use relative units like em or rem for margin, padding, width, and height. here are some sources about CSS units, Relative and Absolute units in CSS and more complex source

  hope this helps

  Marked as helpful

  1