Newsletter sign-up form with success message solution

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted 6 months ago

  Olá Júnior (posso chamar assim?), parabéns pela conclusão de mais um projeto aqui do Frontend Mentor, ficou bem feito.

  Algumas dicas, primeiramente, quanto ao seu CSS, você utilizou medidas relativas no seu card, que foram de min-wdith e max-width, só que como elas são relativas acaba que elas crescem bem mais do que o normal, eu recomendaria utilizar uma medida absoluta, como um width: 920px para desktop por exemplo, que dessa forma ele teria um tamanho fixo e não ficaria crescendo sem parar.

  Outra coisa que observei, é que você utilizou valores relativos também para os ícones, na parte de left: -14%, o melhor seria utilizar também uma medida absoluta, pois dessa forma ele não iria mudar de posição assim que o elemento ou tela crescesse.

  Tanto que no seu @media (min-width: 768px), no seletor main, .sucess-message, eu removi esse trecho e ele ficou mais próximo do design proposto.

  max-width: 50vw;
  max-height: 70vh;
  

  E é isso, parabéns pela conclusão do seu projeto, continue praticando e se aprimorando, caso tenha ficado alguma duvida quanto ao que falei, por favor, comente abaixo que tentarei ajudar da melhor forma possível.

  0