HTML CSS

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  👾Fala André Ximenes, tudo bem? Parabéns pelo seu primeiro desafio e seja bem vinda a comunidade do Frontend Mentor!

  Dei uma olhada aqui na sua solução e você fez tudo muito bem aqui, o design está perfeito.

  O unico problema que voce poderia melhorar é a imagem que está distorcendo na versao desktop, usa object-fit: cover pra fazer ela ficar responsiva e croppar se preciso dentro da colunar.

  Aqui as modificacoes que fiz no seu codigo:

  .banner img {
      display: block;
      /* width: 400px; */
      object-fit: cover;
      max-width: 100%;
      height: 500px;
      position: absolute;
      border-radius: 10px 0 0 10px;
  }
  

  👋 Espero ter ajudado e continue no foco!

  Marked as helpful

  1

  ---

  

  André Ximenes•10

  @andximenesPosted about 2 years ago

  @correlucas Fala Lucas, muito obrigado pelo comentário, com certeza ajudou bastante

  1
• 

  Dawid Kaczmarek•160

  @Dawid2806Posted about 2 years ago

  Hello André I recommend you to learn flex or grid, position is not a good practice in this project

  1

  ---

  

  André Ximenes•10

  @andximenesPosted about 2 years ago

  @Dawid2806 Thanks for the tip, I'll try to learn about flex and grid

  0