Display flex

Community feedback

• 

  Bernardo Poggioni•7,050

  @R3ygoski

  Posted 7 months ago

  Olá Tacia, parabéns pela conclusão do desafio, ele ficou bem-feito.

  Tenho algumas sugestões a dar sobre o seu projeto, primeiro sobre o alinhamento, faltou utilizar min-height: 100vh; no seu seletor body. Dessa forma seu body irá ter 100% do tamanho da viewport, o que significa que ele terá 100% do tamanho da tela visível do navegador.

  A segunda sugestão, é recomendado evitar utilizar CSS Inline, como nesse trecho do seu HTML: <img ... width="290px".../>, isso pode trazer mais dificuldade na manutenção do seu projeto e também deixar levemente desorganizado, especialmente em projetos maiores. Em projetos pequenos o impacto é menor, mas o ideal é sempre manter toda a estilização em apenas um arquivo separado, como você fez no style.css.

  Também percebi que as fontes não seguem as cores do design proposto, para ajustá-las você pode considerar adicionar no seletor main a propriedade: color: hsl(218, 44%, 22%).

  E é isso! Novamente parabéns pela conclusão do desafio, caso tenha ficado qualquer dúvida, por favor comente abaixo que tentarei ajudar da melhor forma possível.

  01

  Marked as helpful
• 

  Simamkele Ndabeni•30

  @VernonVerns

  Posted 7 months ago

  Hi this looks good, well done.

  to align it to the cent of the screen vertically I think you left out align-content/align-items: center; which will vertically align your card.

  00