Newsletter form with two elements and JS

Community feedback

• 

  Grego•1,310

  @Grego14Posted 7 months ago

  Hello! 🎉 congratulations on completing the challenge! 🎉

  It is not necessary to specify properties again in the media query, since it already contains it,

  @media screen and (min-width: 50rem) {
    main {
      /* display: flex; */
      align-items: center;
      justify-content: center;
    }
  }
  

  I do not recommend using units like vh, vw to specify sizes in elements, as it can cause overflow,

  I recommend leaving the alt attribute empty if the image has no semantic meaning or is only for decoration.

  add object-fit: cover property to the image so that it covers the entire space of its parent element.

  I hope this helps! 😁

  Marked as helpful

  1

  ---

  

  KevinBlyweert•170

  @KevinBlyweertPosted 7 months ago

  @Grego14 Thank you ^^ ! I've done some changes, I do not not understand how much I can struggle to make the elements fit perfectly.

  1