Grid, flex, position

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Fala , tudo bem? Parabéns pelo desafio!

  Acabei de ver a sua solução e está muito bem feita, a única coisa que tem pra melhorar é o tamanho da imagem e a posição. Eu mudei o posicionamento da imagem do lado esquerdo porque com o posicionamento que voce colocou estava cropando e também adicionei backgroud-size: cover pra pegar o background por inteiro.

  Além disso, ao invés de usar vh você poder usar px porque vh está relacionado ao tamanho de tela e pixel é relativo a posição do background em si.

  Aqui as correções:

  body {
      height: 100vh;
      background-image: url(../images/bg-pattern-top-desktop.svg), url(../images/bg-pattern-bottom-desktop.svg);
      background-repeat: no-repeat;
      background-size: contain;
      background-position: left -67vh top 0, right -67vh bottom 0;
  }
  

  👋 Espero ter ajudado e continue no foco!

  Marked as helpful

  0
• 

  バレンタイン 😈•64,170

  @VCaramesPosted over 2 years ago

  Congrats on completing this challenge!

  Everything looks good!

  The only thing I would change is your use of pixels; You want to use rem and em.

  By using px, your website isn't fully accessible to the users.

  attached is a link explaining everything in further detail:

  https://engageinteractive.co.uk/blog/em-vs-rem-vs-px

  Marked as helpful

  0