Formulário de inscrição em boletim

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted 7 months ago

  Olá novamente Filipe, parabéns por concluir mais um desafio, o que importa é que você tentou.

  Sobre a parte de responsividade, seu projeto está muito bom na parte de mobile, mas quando chega a 800px de largura ele apresenta alguns problemas, uma forma de resolver isso seria a de retirar o float do seu form. Que dessa forma ele irá subir novamente para dentro do conteúdo. E também adicionar um margin-top: 8rem.

  Outra coisa é mudar o main para display:flex e colocar nele um width: 65vw, e o section colocar um width: 100%; E isso tudo irá fazer ele ficar alinhado somente em algumas telas.

  A noite eu tentarei dar uma modificada no seu projeto e te enviarei no github. Mas uma dica, tente sempre usar o display: flex; pois ele é mais simples de ser utilizado, e isso faz com que você pegue o jeito dele mais rápido. E uma outra coisa que notei, foi que sua estrutura do HTML está um pouco estranha, você colocou tudo dentro de uma section, mas se você reparar bem, verá que na verdade tem duas secções, a primeira é a que está o texto, a lista e o formulário, e a segunda é a que está a imagem, elas são distintas.

  Bom, eu irei dar uma olhada no projeto mais tarde e irei dizer aqui o que consegui fazer.

  E sobre o seu JS, você não está conseguindo voltar do cartão de agradecimento para a página principal, porque você não criou um JavaScript para utilizar com essa segunda página. Uma abordagem que eu recomendo, é a de fazer com que o card principal suma e apareça o de agradecimento, que dessa forma você evita ter duas páginas html.

  E parabéns pela perseverança com o projeto, talvez meu comentário não foi de grande ajuda, mas prometo que irei dar uma olhada nela mais a fundo para tentar te auxiliar da melhor forma possível.

  Marked as helpful

  1

  ---

  

  Filipe Barbosa•90

  @FilipeSBabosaPosted 7 months ago

  Agradeço pela ajuda. Mais não precisa se incomodar tanto dessa vez. Eu tava olhado o projeto de outra pessoa aqui e entendi vários erros, vou tentar melhorar nos próximos projetos @R3ygoski

  1
• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted 7 months ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have a suggestion regarding your code that I believe will be of great interest to you.

  ---

  MINOR TYPO ON FORM LABEL :

  • I have found a minor typo on form label element's for attribute which contains "iemail" as value.

  ---

  • But on name attribute of input element contains mail as value, So in order to fix the typo i would recommend you to change the typo on for and id attribute from "iemail" to "email".

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  1
• 

  Sivaprasath•2,520

  @sivaprasath2004Posted 7 months ago

  Hello i`ll wishing for your completion project solution.

  • I will give some suggestion i believe its used for improve your skill.
  • In mobile site the design was proper but desktop mode the solution was not proper. https://github.com/sivaprasath2004/newsletter-sign-up-form-with-success-message this is my github page by reference you will see what about you will miss the solution code.
  • https://www.frontendmentor.io/solutions/responsive-newsletter-signup-form-with-success-message-_RNJBPoMbv this my challenge solution .please visit what you will improve your solution.

  1