Social proof section solution

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  Fala João, beleza? Acabei de abrir sua solução pelo celular e vi que o background não está pegando a altura inteira da página, um jeito de resolver isso é adicionando min-height:100vh e background-size: cover pra forçar o ajuste.

  Pra alinhar os cards que não pegam a largura inteira vc vai ter que jogar o max-width: 100% e forçar o alinhamento com margin: 0 auto.

  Continue no foco ai 👏

  Marked as helpful

  2

  ---

  

  João Pedro Fernandes•170

  @JPbytePosted over 2 years ago

  @correlucas vlw lucas, tmj bro e obg pelas dicas.

  1
• 

  Adriano•38,270

  @AdrianoEscarabotePosted over 2 years ago

  Eae João, beleza?

  Mandou muito neste desafio, to Vendo que você está bem focado, fazendo desafios todos os dias e evoluindo sempre. Gostei bastante do resultado do desafio, mas tenho algumas dicas que acho que você irá gostar:

  1- O documento deve ter um marco principal, você pode ter colocado todo o conteúdo dentro da tag main, deixar o código mais bem construído. clique aqui para ler sobre

  2- Percebi que o tamanho e as posições dos elementos no layout estava variando muito, uma dica que eu dou para arrumar isso é colocar um max-width, segue o exemplo:

  .collumn { max-width: 1528px; }

  para centralizar o conteudo:

  body { display: flex; flex-direction: collumn; align-items: center; }

  O resto está muito bom!

  Espero que ajude 👍

  Marked as helpful

  1
• 

  David•8,000

  @DavidMorgadePosted over 2 years ago

  Hello Pedro, congrats on finishing the challenge!

  If you don't mind, I would like to give you some suggestions:

  Your layout looks great on deskop and mobile sizes, but for tablet it leaves a lot of space on the right side because by default, your div with the column class is at align-items: flex-start, instead of that try using align-items: center to center the column completely at lower screen sizes!

  For you html, try using more semantic tags, like main, section, article etc... instead of all div to give a more semantically meaning to your content.

  Hope my feedback helps you in future projects, great work!

  Marked as helpful

  1

  ---

  

  João Pedro Fernandes•170

  @JPbytePosted over 2 years ago

  @DavidMorgade thanks bro

  1