Newsletter sign-up form using mobile-first approach

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted 7 months ago

  Olá novamente Isadora, seu projeto está muito bom, ele se assemelha muito ao design proposto, parabéns.

  Só gostaria de fazer algumas notações rápidas, sobre o HTML, sua <div class="signup-img"> poderia ser uma <figure> e sua <div class="signup-info"> poderia ser uma <section> isso é mais recomendável, pois ajuda na acessibilidade.

  E notei que quando deixamos o campo de input vazio e pressionamos para ir para fazer a inscrição, ele não retorna um erro, o correto é ele também retornar um erro dizendo que não pode deixar o campo de input vazio.

  Uma sugestão na sua interpolação de String e Variável, seria a de utilizar uma Template String, pois deixa bem mais simples, mas isso é mais questão de gosto mesmo. Mostrar como ficaria com Template String:

  message.innerHTML =
  `A confirmation email has been sent to <strong class="dark-slate-grey">${email}</strong>. Please open it and click the button inside to confirm your subscription.`
  
  

  Outra coisa que notei foi na questão de redundância, seu código possui duas formas de validar Email, que são validateEmail() e validateInput(), e ambos fazem praticamente a mesma coisa pelo que identifiquei.

  Por fim, para facilitar a compreensão da parte relacionada ao JavaScript, fiz o download do seu código e realizei algumas modificações, adicionando comentários explicativos. Estou planejando abrir um Issue no seu repositório do GitHub em breve, apontando para as alterações que fiz.

  Novamente parabéns pelo seu projeto, ele está muito bom mesmo, continue praticando e se aprimorando. Caso algo que eu tenha dito não tenha ficado claro, você já sabe.

  Marked as helpful

  0