Responsive landing page using flexbox

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 2 years ago

  Hi @mati0165 👋, good job completing this challenge, and welcome to the Frontend Mentor Community! 🎉

  It's a great solution and a good start. Here are a few suggestions I've made that you can consider in the future if you're looking to improve the solution further.

  • Update the image selector to make responsive images.

  img {
      /* width: 16rem; */
      /* max-height: 18rem; */
      width: 100%;
      object-fit: contain;
      border-radius: 0.625rem;
  }
  

  • You could use a CSS Reset to remove browser built-in styles and reduce browser inconsistencies. Your body's default margin causes the vertical scrollbar to appear.

  Popular reset style sheets:

  • A modern alternative to CSS resets
  • CSS Tools: Reset CSS
  • Custom CSS Reset

  Overall well done! I hope those tips will help you.

  Good job, and happy coding!

  Marked as helpful

  1